1043111 - Allow mState==DECODER_STATE_DORMANT in MediaDecoderStateMachine::ResetPlayback()

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Description

[JW Wang [:jwwang] [:jw_wang]]

When FlushDecoding() [1] is called in dormant state, it will call ResetPlayback() and the assertion will fail at [2]. We should allow |mState==DECODER_STATE_DORMANT| in the assertion.


[1] http://hg.mozilla.org/mozilla-central/file/30907d52c4c2/content/media/MediaDecoderStateMachine.cpp#l2294

[2] http://hg.mozilla.org/mozilla-central/file/30907d52c4c2/content/media/MediaDecoderStateMachine.cpp#l1408

Comment 1

[JW Wang [:jwwang] [:jw_wang]]

Attachment: 1043111_fix_ResetPlayback_in_dormant.patch

See comment 0 for the root cause.

Comment 2

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/b97c05712090

Comment 3

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/b97c05712090