Closed Bug 1361944 Opened 3 years ago Closed 3 years ago

ASSERTION: Shouldn't have a decoder: 'mDecoder == nullptr'

Categories

(Core :: Audio/Video: Playback, enhancement, P1)

enhancement

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: jwwang, Assigned: jwwang)

References

Details

Attachments

(2 files)

Assignee: nobody → jwwang
Priority: -- → P1
See Also: → 1361756
Attachment #8864397 - Flags: review?(cpearce)
Attachment #8864398 - Flags: review?(cpearce)
Comment on attachment 8864398 [details]
Bug 1361944. P2 - be sure to shut down the existing decoder before loading the next source child.

https://reviewboard.mozilla.org/r/136088/#review139460
Attachment #8864398 - Flags: review?(cpearce) → review+
Comment on attachment 8864397 [details]
Bug 1361944. P1 - add a crash test.

https://reviewboard.mozilla.org/r/136086/#review139462
Attachment #8864397 - Flags: review?(cpearce) → review+
Thanks!
Pushed by jwwang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/763cc804a25d
P1 - add a crash test. r=cpearce
https://hg.mozilla.org/integration/autoland/rev/beace30c1073
P2 - be sure to shut down the existing decoder before loading the next source child. r=cpearce
I will disable the test on Android.
Hi Sheriff,
Can you back out only P1 from the tree? Thanks!
backed out P1 for causing crashtest perma failure on Android like https://treeherder.mozilla.org/logviewer.html#?job_id=96808742&repo=autoland&lineNumber=3888
Backout by ihsiao@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9316a48f009b
Backed out changeset 763cc804a25d for causing crashtest perma failure on Android
https://hg.mozilla.org/mozilla-central/rev/beace30c1073
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Blocks: 1362793
You need to log in before you can comment on or make changes to this bug.