Closed Bug 1332149 Opened 3 years ago Closed 3 years ago

Don't expose plugin-container or sandbox vouchers to GMPs.

Categories

(Core :: Audio/Video: GMP, defect, P3)

defect

Tracking

()

RESOLVED FIXED
mozilla53
Tracking Status
firefox53 --- fixed

People

(Reporter: cpearce, Assigned: cpearce)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Since we're removing Adobe Primetime, we no longer need to expose the plugin-container or plugin vouchers to GMPs.
Comment on attachment 8828262 [details]
Bug 1332149 - Don't expose plugin-container or sandbox vouchers to GMPs.

https://reviewboard.mozilla.org/r/105736/#review106622

r+ with nit:

::: dom/media/gmp/gmp-api/gmp-decryption.h:231
(Diff revision 1)
>  
>  // API exposed by plugin library to manage decryption sessions.
>  // When the Host requests this by calling GMPGetAPIFunc().
>  //
>  // API name macro: GMP_API_DECRYPTOR
>  // Host API: GMPDecryptorHost

I think you can remove this line too.
Attachment #8828262 - Flags: review?(gsquelart) → review+
Pushed by cpearce@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/79e4c3c3f793
Don't expose plugin-container or sandbox vouchers to GMPs. r=gerald
https://hg.mozilla.org/mozilla-central/rev/79e4c3c3f793
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.