Closed
Bug 1113282
Opened 9 years ago
Closed 9 years ago
Crash in MSE conformance test suite due to TrackBuffer initialization racing with shutdown
Categories
(Core :: Audio/Video, defect)
Tracking
()
RESOLVED
FIXED
mozilla37
People
(Reporter: bholley, Assigned: bholley)
References
Details
Attachments
(1 file)
6.06 KB,
patch
|
cajbir
:
review+
Sylvestre
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
I get a failed MOZ_ASSERT on the AbortVideo tests of [1]. I've got a patch, will push to try. [1] http://yt-dash-mse-test.commondatastorage.googleapis.com/unit-tests/2015.html?timestamp=1418928742765
Assignee | ||
Comment 1•9 years ago
|
||
https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=5748e8eecfe9
Assignee | ||
Comment 2•9 years ago
|
||
Attachment #8538733 -
Flags: review?(cajbir.bugzilla)
Updated•9 years ago
|
Attachment #8538733 -
Flags: review?(cajbir.bugzilla) → review+
Assignee | ||
Comment 3•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/4757a1d4a23e
Comment 4•9 years ago
|
||
sorry backedout and found out this change was innocent for causing https://treeherder.mozilla.org/logviewer.html#?job_id=4797293&repo=mozilla-inbound so relanded in https://hg.mozilla.org/integration/mozilla-inbound/rev/e7dbd3eb21e5
Comment 5•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/e7dbd3eb21e5
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
Comment 7•9 years ago
|
||
Comment on attachment 8538733 [details] [diff] [review] Check during async TrackBuffer decoder initialization to make sure we haven't been shut down. v1 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]: Low. Affects non-MSE code but is well tested by now. [String/UUID change made/needed]: None.
Attachment #8538733 -
Flags: approval-mozilla-aurora?
Updated•9 years ago
|
status-firefox36:
--- → affected
status-firefox37:
--- → fixed
Updated•9 years ago
|
Attachment #8538733 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in
before you can comment on or make changes to this bug.
Description
•