1172387 - Clean up MediaDecoderStateMachine::StopAudioThread

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Description

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

StopAudioThread() is now called in state machine thread only. We can now remove |mStopAudioThread| which is used to handle reentrancy on decoder thread.

Comment 1

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

Attachment: 1172387_cleanup_StopAudioThread-v1.patch

Comment 2

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

Thanks for the review.

Comment 3

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

Try looks green. https://treeherder.mozilla.org/#/jobs?repo=try&revision=52b59eeef3ba

Comment 4

[Pulsebot]

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

Comment 5

[Ryan VanderMeulen [:RyanVM]]

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