Assertion failure: mSrcStreamPausedGraphTime == GRAPH_TIME_MAX, at /builds/worker/workspace/build/src/dom/html/HTMLMediaElement.cpp:4665
Categories
(Core :: WebRTC: Audio/Video, defect, P2)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr60 | --- | unaffected |
firefox64 | --- | unaffected |
firefox65 | --- | wontfix |
firefox66 | --- | fixed |
People
(Reporter: jkratzer, Assigned: pehrsons)
References
(Blocks 1 open bug)
Details
(Keywords: assertion, testcase)
Attachments
(3 files)
(deleted),
text/html
|
Details | |
(deleted),
application/x-javascript
|
Details | |
(deleted),
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta-
|
Details |
Comment 1•6 years ago
|
||
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Comment 2•6 years ago
|
||
Reporter | ||
Comment 3•6 years ago
|
||
Updated•6 years ago
|
Comment 4•6 years ago
|
||
Comment 5•6 years ago
|
||
Assignee | ||
Comment 6•6 years ago
|
||
Assignee | ||
Comment 7•6 years ago
|
||
Assignee | ||
Comment 8•6 years ago
|
||
Assignee | ||
Comment 9•6 years ago
|
||
Assignee | ||
Updated•6 years ago
|
Comment 10•6 years ago
|
||
Assignee | ||
Comment 11•6 years ago
|
||
Comment on attachment 9034430 [details]
Bug 1515068 - Allow UpdateSrcStreamTime while paused. r?jya
[Beta/Release Uplift Approval Request]
Feature/Bug causing the regression: Bug 1423241
User impact if declined: Debug build assertion failures
Is this code covered by automated tests?: No
Has the fix been verified in Nightly?: No
Needs manual test from QE?: No
If yes, steps to reproduce:
List of other uplifts needed: None
Risk to taking this patch: Low
Why is the change risky/not risky? (and alternatives if risky): Removes a few assertions that can legitimately happen. There's a change to logic too, but it's trivial and simply ignores redundant currentTime updates.
String changes made/needed:
Comment 12•6 years ago
|
||
bugherder |
Comment 13•6 years ago
|
||
Comment on attachment 9034430 [details]
Bug 1515068 - Allow UpdateSrcStreamTime while paused. r?jya
Given that there's no real user impact here outside of debug builds, I think this can just ride the trains.
Updated•6 years ago
|
Description
•