Skip to content
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

Fix normative notes and clarify/simplify robustness rules #549

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

joeyparrish
Copy link
Member

@joeyparrish joeyparrish commented Jun 18, 2024

Issue #521

This clarifies current reality in implementation, and may be enough to close the issue on that basis. But I plan to propose one small behavioral change after this clarification, which there may or may not be consensus on.


Preview | Diff

Comment on lines +2548 to +2549
levels specified in the configuration of the {{MediaKeySystemAccess}} object used
to create the {{MediaKeys}} object.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
levels specified in the configuration of the {{MediaKeySystemAccess}} object used
to create the {{MediaKeys}} object.
levels specified in the configuration of the {{MediaKeySystemAccess}} dictionary used
to create the {{MediaKeys}} instance.

Comment on lines +2558 to +2559
Applications are recommended to specify the robustness level(s) they require,
if any, to avoid unexpected client incompatibilities.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Applications are recommended to specify the robustness level(s) they require,
if any, to avoid unexpected client incompatibilities.
To avoid unexpected interoperability issues, developers are encouraged to provide the robustness level(s)
(if any).

Copy link
Member

@marcoscaceres marcoscaceres left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couple of wording suggestions... I'm not very familiar with how EME works, so would need someone else to check the technical details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants