Closed
Bug 1277136
Opened 5 years ago
Closed 5 years ago
Reset GMP script should run in content context
Categories
(Testing Graveyard :: external-media-tests, defect)
Testing Graveyard
external-media-tests
Tracking
(firefox49 fixed)
RESOLVED
FIXED
mozilla49
Tracking | Status | |
---|---|---|
firefox49 | --- | fixed |
People
(Reporter: bryce, Assigned: bryce)
Details
Attachments
(1 file)
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.
Assignee | ||
Updated•5 years ago
|
Assignee: nobody → bvandyk
Assignee | ||
Comment 1•5 years ago
|
||
- 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/
Attachment #8758521 -
Flags: review?(mjzffr)
Updated•5 years ago
|
Summary: Reset GMP script should run on context thread → Reset GMP script should run in content context
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
Attachment #8758521 -
Flags: review?(mjzffr) → review+
Assignee | ||
Updated•5 years ago
|
Keywords: checkin-needed
Pushed by cpearce@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/23fde8ecfa91 External Media Tests: Fix typos, update context usage, update gmp reset script. r=maja_zf
Keywords: checkin-needed
Comment 4•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/23fde8ecfa91
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox49:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
Updated•3 years ago
|
Product: Testing → Testing Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•