1277136 - Reset GMP script should run in content context

Status: RESOLVED FIXED

(Testing Graveyard :: external-media-tests, defect)

Description

[Bryce Seager van Dyk [:bryce] (he/him) - Not reading bugmail]

The script in EMESetupMixin (https://dxr.mozilla.org/mozilla-central/source/dom/media/test/external/external_media_harness/testcase.py#214) that resets GMP is currently being called in the chrome context. This needs to be done in the content context so that the appropriate handlers are notified.

Comment 1

[Bryce Seager van Dyk [:bryce] (he/him) - Not reading bugmail]

Attachment: MozReview Request: Bug 1277136 - External Media Tests: Fix typos, update context usage, update gmp reset script. r?maja_zf

- Fix a typo in media.gmp-eme-adobe.version pref strings.
- Update reset gmp script call to be from content context. This
would fail if done from the chrome process.
- Update reset gmp script to use new requestMediaKeySystemAccess
syntax.

Review commit: https://reviewboard.mozilla.org/r/56796/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/56796/

Comment 2

[Maja Frydrychowicz :impossibus (was :maja_zf) (needinfo me)]

Comment on attachment 8758521 [details]
MozReview Request: Bug 1277136 - External Media Tests: Fix typos, update context usage, update gmp reset script. r?maja_zf

https://reviewboard.mozilla.org/r/56796/#review53590

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/23fde8ecfa91
External Media Tests: Fix typos, update context usage, update gmp reset script. r=maja_zf

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/23fde8ecfa91