Closed Bug 1000813 Opened 10 years ago Closed 10 years ago

WARNING: NS_ENSURE_TRUE(mDecoderStateMachine) failed: file /builds/slave/try-lx-d-000000000000000000000/build/content/media/MediaDecoder.cpp, line 1025

Categories

(Core :: Audio/Video, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla32

People

(Reporter: jwwang, Assigned: jwwang)

Details

Attachments

(1 file)

https://tbpl.mozilla.org/php/getParsedLog.php?id=38372921&tree=Try&full=1

Found this warning when running test_seek.html.

MediaDecoder::NotifyBytesConsumed() is called by DispatchBytesConsumedEvent::Run() which could run later than the async shutdown process of MediaDecoder and MediaDecoderStateMachine.

We should just return from NotifyBytesConsumed() if |mShuttingDown| is true without checking |mDecoderStateMachine|.
See comment 0.
Assignee: nobody → jwwang
Status: NEW → ASSIGNED
Attachment #8420729 - Flags: feedback?(rlin)
Attachment #8420729 - Flags: feedback?(rlin) → feedback+
Attachment #8420729 - Flags: review?(cpearce)
Attachment #8420729 - Flags: review?(cpearce) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/4f35482325db
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: