[EME] Widevine fails on Shaka player demo with DRM.REQUESTED_KEY_SYSTEMS_UNAVAILABLE

RESOLVED FIXED in Firefox 49

Status

()

Core
Audio/Video: Playback
P1
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: cpearce, Assigned: cpearce)

Tracking

unspecified
mozilla49
Points:
---

Firefox Tracking Flags

(firefox49 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
Shaka player demo is failing on Widevine content with error DRM.REQUESTED_KEY_SYSTEMS_UNAVAILABLE.

This is a regression from bug 1271242, which thankfully hasn't been uplifted - and doesn't need to be.

The problem is that I made EMEDecoderModule::SupportsMimeType() call GMPDecoderModule::SupportsMimeType() specifying a GMP to use, but I didn't add the widevine keysystem to the take of keysystems that we recognise in GMPDecoderModule. So we're unable to create an EME MediaDataDecoder.
(Assignee)

Comment 1

2 years ago
Created attachment 8751114 [details]
MozReview Request: Bug 1271883 - Add Widevine keysystem to sGMPCodecs. r?jwwang

I broke Widevine in 1271242 when I changed EMEDecoderModule::SupportsMimeType()
to call GMPDecoderModule::SupportsMimeType() specifying a GMP to use, as I
didn't add the widevine keysystem to the take of keysystems that we recognise
in GMPDecoderModule. So we're unable to create an EME MediaDataDecoder.

In order to make Widevine EME work again, we need the Widevine keysystem string
in sGMPCodecs in GMPDecoderModule.cpp.

Review commit: https://reviewboard.mozilla.org/r/51833/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/51833/
Attachment #8751114 - Flags: review?(jwwang)
Comment on attachment 8751114 [details]
MozReview Request: Bug 1271883 - Add Widevine keysystem to sGMPCodecs. r?jwwang

https://reviewboard.mozilla.org/r/51833/#review48673
Attachment #8751114 - Flags: review?(jwwang) → review+

Comment 3

2 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/f173149e870d

Comment 4

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/f173149e870d
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox49: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in before you can comment on or make changes to this bug.