Closed
Bug 1109389
Opened 9 years ago
Closed 9 years ago
Assertions in MediaTaskQueue when shutting down EMEDecryptor in mochitest
Categories
(Core :: Audio/Video, defect)
Tracking
()
RESOLVED
FIXED
mozilla37
People
(Reporter: cpearce, Assigned: cpearce)
References
(Blocks 1 open bug)
Details
Attachments
(1 file, 1 obsolete file)
10.20 KB,
patch
|
cpearce
:
review+
Sylvestre
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
When running mochitests dom/media/test/test_eme* I get assertion failures: [4380] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80004005: file c:\Users\cpearce\src\mozilla\purple\dom\media\MediaTaskQueue.cpp, line 106 [4380] WARNING: NS_ENSURE_SUCCESS(rv, rv) failed with result 0x80004005: file c:\Users\cpearce\src\mozilla\purple\dom\media\MediaTaskQueue.cpp, line 106 This is because the EME PDM creates a task queue which is shares with the EMEDecryptors that it creates. So if an EME PDM creates two EMEDecryptors (say one for the video and another for the audio streams...) when the first EMEDecryptor shutsdown, it will shutdown the task queue, and the second EMEDecryptor won't be able to use it to shutdown. The failure to use the task queue is why we see those assertion failures.
Assignee | ||
Comment 1•9 years ago
|
||
Have the constructor of EMEDecryptor create its own task queue.
Attachment #8534060 -
Flags: review?(kinetik)
Assignee | ||
Comment 2•9 years ago
|
||
https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=ea53cf992be6 https://tbpl.mozilla.org/?tree=Try&rev=ea53cf992be6
Updated•9 years ago
|
Attachment #8534060 -
Flags: review?(kinetik) → review+
Assignee | ||
Comment 3•9 years ago
|
||
https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=16dacd65d641 https://tbpl.mozilla.org/?tree=Try&rev=16dacd65d641
Assignee | ||
Comment 4•9 years ago
|
||
Because life was too easy before WAE was turned on: https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=6bbcc2e5289a
Assignee | ||
Comment 5•9 years ago
|
||
Fixed WAE productivity suck. https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=6bbcc2e5289a Carry forward r=kinetik.
Attachment #8534060 -
Attachment is obsolete: true
Attachment #8534127 -
Flags: review+
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Comment 6•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/dfcd18d91159
Keywords: checkin-needed
Comment 7•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/dfcd18d91159
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
Comment 8•9 years ago
|
||
Comment on attachment 8534127 [details] [diff] [review] Patch: Store GMP record name in record file on disk Approval Request Comment [Feature/regressing bug #]: MSE [User impact if declined]: less consistent testing, sites more likely to serve flash video. [Describe test coverage new/current, TBPL]: landed on m-c. [Risks and why]: I've not been backporting EME-related changes, so this may end up requiring other related fixes. However, EME itself is still pref'd-off in Aurora, and this is an isolated change, so I'd say the risk is low. [String/UUID change made/needed]: none. The MSE-related patches in bug 1110534 depend on this change.
Attachment #8534127 -
Flags: approval-mozilla-aurora?
Updated•9 years ago
|
status-firefox36:
--- → affected
status-firefox37:
--- → fixed
Updated•9 years ago
|
Attachment #8534127 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 10•9 years ago
|
||
Mass update firefox-status to track EME uplift.
status-firefox38:
--- → fixed
status-firefox39:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•