w3ctag / design-reviews

W3C specs and API reviews
Creative Commons Zero v1.0 Universal
332 stars 56 forks source link

Support codec and container switching with MSE using SourceBuffer.changeType() #298

Closed wolenetz closed 6 years ago

wolenetz commented 6 years ago

Bonjour TAG,

I'm requesting a TAG review of:

Further details (optional):

Firefox 63+ contains an implementation of this new API (SourceBuffer.changeType()) on-by-default. Chrome 69+ contains an implementation conditioned by chrome://flags#enable-experimental-web-platform-features, and the intent-to-ship it on-by-default is currently pending approvals.

Please go by the editor's draft, not the explainer (the explainer predated the spec, and some small things have changed such as changeType() is a void, not boolean method; it throws if called in bad state or if the change isn't supported.) I'll be updating the explainer this week to make it reflect the spec more precisely.

We'd prefer the TAG provide feedback as (please select one):

Thank you!

travisleithead commented 6 years ago

I will note that this has lots of support from various teams inside of Microsoft. :)

wolenetz commented 6 years ago

Safari Technical Preview, as of at least a week ago, also contained an experimental implementation of SourceBuffer.changeType(). @jernoble

cynthia commented 6 years ago

We're happy to see this addition (bug fix?) to the feature set of MSE. As a much needed addition to the currently shipping MSE API, I don't think we have any notable objections towards the design or functionality of the proposal.

(I do state we, but I haven't actually discussed this with the rest of the group yet. To be brought up in a upcoming call.)

travisleithead commented 6 years ago

Likewise, have reviewed this, and have no outstanding concerns. Really appreciate the detailed explainer and concerns and design alternatives considered.

cynthia commented 6 years ago

Taken up at Paris F2F, we're happy to see this work move forward - thanks for bringing this to our attention.