1227790 - [EME] Update MediaKeyStatus WebIDL enum to include "released", "output-restricted" and "status-pending"

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, defect)

Description

[Chris Pearce [:cpearce (Not reading bugmail)]]

The specification for the MediaKeyStatus enumeration has changed. We should update to the current spec.

We will also need to change enumeration that we use to represent these in the GMP API. We need to be careful, because if a new Adobe GMP that uses the new larger GMP API enumeration tries to send one of the new values in an IPC message to an old version of Firefox that expects the old enumerated valued, the IPDL serialization of the enum value will fail, killing the GMP process.

So we need to be sure to not push a new Adobe GMP that uses the new values out to old versions of Firefox which use the new enum values.

Comment 1

[Chris Pearce [:cpearce (Not reading bugmail)]]

Attachment: Patch 1: Update MediaKeyStatuses to include "released", "output-restricted" and "status-pending".

Update MediaKeyStatus enum to match draft EME spec.

Comment 2

[Chris Pearce [:cpearce (Not reading bugmail)]]

Attachment: Patch 2: Update GMP API to use new MediaKeyStatus enum types

Add the new MediaKeyStatus enum types to the end of the corresponding enum in the GMP API, and fixup the GMP enum to DOM enum helper function.

Comment 3

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 8691673 [details] [diff] [review]
Patch 1: Update MediaKeyStatuses to include "released", "output-restricted" and "status-pending".

So the "output-not-allowed" value is just gone, right?

r=me

Comment 4

[Chris Pearce [:cpearce (Not reading bugmail)]]

(In reply to Boris Zbarsky [:bz] from comment #3)
> So the "output-not-allowed" value is just gone, right?

Yup.

Comment 5

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/c9162d1acbe4
https://hg.mozilla.org/integration/mozilla-inbound/rev/5c0a41a39290

Comment 6

[Chris Pearce [:cpearce (Not reading bugmail)]]

Consider uplifing.

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/c9162d1acbe4
https://hg.mozilla.org/mozilla-central/rev/5c0a41a39290