-
Notifications
You must be signed in to change notification settings - Fork 80
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add ISO Common Encryption 'cbcs' Protection Scheme. Fixes #391. #392
Conversation
I don't know why the IPR check failed. I am in the HME WG. |
The HME WG's charter expired on Apr 30 so until the charter is extended no one is actually a member of the HME WG. ;<) @plehegar needs to fix this. /paulc |
Adding a protection scheme would probably impact the tests, right? If yes, then this seems a late request imho. |
This change brings the spec more into alignment with shipping implementations. For instance, Safari has supported |
@hober We weren't able to run the test suite with Safari back last year. Have things changed and Safari is now spec-compliant ? |
@mwatson2 WebKit is working on improving our compatibility with the test suite. But apart from bringing our DOM APIs up to spec, we have the additional limitation on macOS that our platform only supports the 'cbcs' scheme. Hence our desire to document Safari's shipping behavior. |
What's the rationale for deferring this to v.next? |
EME V1 is done and the Proposed Recommendation is being dealt with by the Director and Advisory Committee. IFF EME becomes a Recommendation the current HME WG charter would only permit us to work on non-breaking changes errata. /paulc |
Fair enough. Any reason to not spin up an Editor's Draft of small changes like this one, so that we can hit the ground running when the way forward is clearer? |
Note that this pull request actually proposes changes to the non-normative Encrypted Media Extensions Stream Format Registry which was last published as a WG Note in Sep 2016. I believe this kind of change can be made at any time IFF we can get consensus on the change. /paulc |
Any objections to merging this now? I'll be sending a PR soon for an API change to query encryption scheme support, including 'cbcs', and this seems like a needed compliment to that. |
@joeyparrish I'm all for it. |
I'll go ahead and merge it, then. I may make a few edits on top of that as I work to complete PR #457. |
Thanks! |
No description provided.