Define MediaCapabilitiesDecodingInfo.codecSwitchingSupported #165
Conversation
|
@ grabbing some stakeholders (others welcome): @jernoble @xhwang-chromium @joeyparrish @gregwfreedman Please let me know your thoughts! |
|
I don't know how to comment on the diff directly, so I put my comments here. For
Should it be For
What if the pipeline only supports switching to some codec, but not others? I think we agreed to keep the API stateless for simplicity. In that case, does it make sense to only set |
|
Thanks for reviewing!
Yes, done!
Yes, thats my intent and I've amended the language to reflect this. The API gets a lot more complicated if we need to describe a subset of codecs where transitions are supported. |
| @@ -94,6 +100,8 @@ spec: encrypted-media; for: EME; urlPrefix: https://www.w3.org/TR/encrypted-medi | |||
| text: MediaKeysRequirement; url: dom-mediakeysrequirement | |||
| text: audioCapabilities; url: dom-mediakeysystemconfiguration-audiocapabilities | |||
| text: contentType; url: dom-mediakeysystemmediacapability-contenttype | |||
| type: method | |||
There was a problem hiding this comment.
You shouldn't need this in theory. Or if we do, we need to figure out why it hasn't been exported from the other spec.
| <p> | ||
| A {{MediaCapabilitiesDecodingInfo}} has an associated | ||
| <dfn dict-member for='MediaCapabilitiesDecodingInfo'> | ||
| codecSwitchingSupported</dfn> which is a boolean. |
There was a problem hiding this comment.
| codecSwitchingSupported</dfn> which is a boolean. | |
| codecSwitchingSupported</dfn> that is a boolean. |
| @@ -940,6 +980,23 @@ spec: secure-contexts; urlPrefix: https://www.w3.org/TR/secure-contexts/ | |||
| efficiency unless the device's power source has side effects | |||
| such as enabling different decoding modules. | |||
| </li> | |||
| <li> | |||
| If <code>configuration.type</code> is set to | |||
There was a problem hiding this comment.
can type be linked to something?
Implements the latest proposal from the discussion in #102. This adds a single
codecSwitchingSupportedboolean to the MediaCapabilitiesDecodingInfo dictionary that indicates whether sourceBuffer.changeType() may be used to switch codecs within the "decoding pipeline" as described by the config passed to mediaCapabilties.decodingInfo().This PR creates a definition of "decoding pipeline" which distinguishes between clear vs eme playback (and eme playback of differently named key systems). It may occur that a given UA does not really have a separate pipeline for clear vs its platform key system, but this is still allowed by the specification as that difference is not observable.
💥 Error: connect ETIMEDOUT 173.230.149.95:443 💥
PR Preview failed to build. (Last tried on Jan 9, 2024, 11:53 AM UTC).
More
PR Preview relies on a number of web services to run. There seems to be an issue with the following one:
🚨 CSS Spec Preprocessor - CSS Spec Preprocessor is the web service used to build Bikeshed specs.
🔗 Related URL
If you don't have enough information above to solve the error by yourself (or to understand to which web service the error is related to, if any), please file an issue.