Closed
Bug 1124939
Opened 9 years ago
Closed 9 years ago
[EME] Add "individualization-request" to MediaKeyMessageType
Categories
(Core :: Audio/Video, defect)
Tracking
()
RESOLVED
FIXED
mozilla38
People
(Reporter: cpearce, Assigned: jwwang)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
3.84 KB,
patch
|
bzbarsky
:
review+
|
Details | Diff | Splinter Review |
3.97 KB,
patch
|
lmandel
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
https://github.com/w3c/encrypted-media/commit/1a611eab4f93318e4bba4207bbcba673d42cd062 adds "individualization-request" to MediaKeyMessageType. We'll need to update to include that.
Assignee | ||
Comment 3•9 years ago
|
||
Do I need to change "eme-decrypt-v4" to "eme-decrypt-v5" for just adding a enum field?
Flags: needinfo?(cpearce)
Reporter | ||
Comment 4•9 years ago
|
||
Yes, otherwise when a GMP that uses the new enum tries to use the enum, the IPC serialization could fail in a v4 Gecko without the new enum value.
Flags: needinfo?(cpearce)
Assignee | ||
Comment 5•9 years ago
|
||
Ah, I see. Thanks for the reply.
Assignee | ||
Comment 6•9 years ago
|
||
Add "individualization-request" to MediaKeyMessageType.
Attachment #8555022 -
Flags: review?(cpearce)
Reporter | ||
Comment 7•9 years ago
|
||
Comment on attachment 8555022 [details] [diff] [review] 1124939_add_individualization_request-v1.patch Review of attachment 8555022 [details] [diff] [review]: ----------------------------------------------------------------- Will need a DOM peer signoff on the WebIDL change here too.
Attachment #8555022 -
Flags: review?(cpearce) → review+
Assignee | ||
Comment 8•9 years ago
|
||
Comment on attachment 8555022 [details] [diff] [review] 1124939_add_individualization_request-v1.patch Hi bz, Can you reivew the webidl file? Thanks.
Attachment #8555022 -
Flags: review+ → review?(bzbarsky)
Comment 9•9 years ago
|
||
Comment on attachment 8555022 [details] [diff] [review] 1124939_add_individualization_request-v1.patch r=me
Attachment #8555022 -
Flags: review?(bzbarsky) → review+
Assignee | ||
Comment 10•9 years ago
|
||
Try: https://tbpl.mozilla.org/?tree=Try&rev=69be7315a89c
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Comment 11•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/ee1ddfd22521
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/ee1ddfd22521
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
Reporter | ||
Updated•9 years ago
|
Blocks: eme-platform-uplift
Reporter | ||
Comment 13•9 years ago
|
||
https://hg.mozilla.org/releases/mozilla-beta/rev/6f83d3fe38da
status-firefox37:
--- → fixed
Reporter | ||
Comment 14•9 years ago
|
||
Patch for beta branch as part of EME platform uplift.
Reporter | ||
Comment 15•9 years ago
|
||
Comment on attachment 8572323 [details] [diff] [review] Beta patch Requesting retroactive approval for Beta landing as part of EME platform uplift.
Attachment #8572323 -
Flags: approval-mozilla-beta?
Updated•9 years ago
|
status-firefox38:
--- → fixed
Comment 16•9 years ago
|
||
Comment on attachment 8572323 [details] [diff] [review] Beta patch Approved for Beta as part of EME platform uplift.
Attachment #8572323 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in
before you can comment on or make changes to this bug.
Description
•