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

FF129 MediaCapabilities.decodingInfo() - encrypted content #23815

Merged
merged 3 commits into from
Jul 30, 2024

Conversation

hamishwillee
Copy link
Contributor

@hamishwillee hamishwillee commented Jul 15, 2024

FF129 Adds support for using MediaCapabilities.decodingInfo() with encoded media in https://bugzilla.mozilla.org/show_bug.cgi?id=1898344

The spec is a bit messy, but what it means is that

  • the configuration object passed to the decodingInfo() method has a new property called keySystemConfiguration.
  • The returned promise resolves to include the property keySystemAccess which is a MediaKeySystemAccess you can use to get keys and decode the content for playback.

What I have done is added a subfeature under decodingInfo() for the keySystemConfiguration parameter. I haven't added another subfeature for the keySystemAccess in the returned object - should I add that too, even though it would be identical?

Or is to have just one entry but name it something like encrypted_media?

Related docs work can be tracked in mdn/content#34696

Note: The spec has another value configuration in the return object, but that is not implemented by anyone.

@github-actions github-actions bot added the data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Jul 15, 2024
@hamishwillee hamishwillee force-pushed the ff129_mediacapabilities.decodeInfo_encr branch from 315b125 to f4e2b90 Compare July 19, 2024 03:06
@hamishwillee
Copy link
Contributor Author

@Elchi3 Could I please have a review on this?

Copy link
Member

@Elchi3 Elchi3 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks Hamish! 👍

@Elchi3 Elchi3 merged commit 81e0259 into mdn:main Jul 30, 2024
6 checks passed
@hamishwillee hamishwillee deleted the ff129_mediacapabilities.decodeInfo_encr branch July 30, 2024 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants