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

EME MediaKeySessionClosedReason #671

Closed
1 task done
xhwang-chromium opened this issue Aug 27, 2021 · 1 comment
Closed
1 task done

EME MediaKeySessionClosedReason #671

xhwang-chromium opened this issue Aug 27, 2021 · 1 comment

Comments

@xhwang-chromium
Copy link

Ya ya yawm TAG!

I'm requesting a TAG review of EME MediaKeySessionClosedReason.

The current EME spec says:

"the CDM may close a session at any point, such as when the session is no longer needed or when system resources are lost."

However, there's no way to specify the exact reason for session closure. In some cases, this is part of normal user flow, e.g. user close laptop lid to put the device into sleep mode, where the player should resume playback. In some other cases, this is due to some unrecoverable fatal error.

This proposal updates the closed attribute on MediaKeySession to provide a MediaKeySessionClosedReason so that the JavaScript player can handle session closure differently based on the reasons.

Further details:

  • I have reviewed the TAG's Web Platform Design Principles
  • Relevant time constraints or deadlines: I'd like to ship this feature by default in Chromium M96.
  • The group where the work on this specification is currently being done: https://www.w3.org/media-wg/
  • The group where standardization of this work is intended to be done (if current group is a community group or other incubation venue): https://www.w3.org/media-wg/
  • Major unresolved issues with or opposition to this specification: None
  • This work is being funded by: N/A

We'd prefer the TAG provide feedback as (please delete all but the desired option):

💬 leave review feedback as a comment in this issue and @-notify @xhwang-chromium

@cynthia
Copy link
Member

cynthia commented Sep 16, 2021

(Setting aside all debate about whether or not EME should exist in the first place, since the cat is out of the box. For that bit, please refer to our feedback from 2017.)

Given that this is a small incremental change to an existing spec that improves developer ergonomics, we are happy to see this move forward. Thanks for bringing this to our attention.

Noticed that the EME spec doesn't have a S&P section - think it would be worth adding something there (not just for this feature, but for EME as a whole) if possible.

@cynthia cynthia closed this as completed Sep 16, 2021
@torgo torgo added the Resolution: satisfied The TAG is satisfied with this design label Sep 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants