1328810 - Remove the call to DispatchDecodeTasksIfNeeded() from MDSM::StopPlayback()

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, defect, P3)

Description

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

http://searchfox.org/mozilla-central/rev/82ebc0c5ea45477ef4a4bfb3bbff35069bd18cee/dom/media/MediaDecoderStateMachine.cpp#2730

When we stop playback, we stop consuming media data. There is no need for decoding when playback stops.

Comment 1

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

Attachment: Bug 1328810 - Remove the call to DispatchDecodeTasksIfNeeded() from MDSM::StopPlayback().

Review commit: https://reviewboard.mozilla.org/r/102812/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/102812/

Comment 2

[Tzuhao Kuo [:kaku]]

Comment on attachment 8824290 [details]
Bug 1328810 - Remove the call to DispatchDecodeTasksIfNeeded() from MDSM::StopPlayback().

https://reviewboard.mozilla.org/r/102812/#review103320

Comment 3

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

Thanks!

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6bd36352070b
Remove the call to DispatchDecodeTasksIfNeeded() from MDSM::StopPlayback(). r=kaku

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/6bd36352070b