+++ This bug was initially created as a clone of Bug #1121064 +++ Call DestroyDecodedStream() in MediaDecoder::SetDormantIfNecessary() seem not a correct way to stop MediaStream in this situation.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=fe2639b3b10b https://tbpl.mozilla.org/?tree=Try&rev=fe2639b3b10b

FYI: Diagram related to HTMLMediaElement::MozCaptureStream() https://github.com/sotaroikeda/firefox-diagrams/blob/master/media/dom_media_HTMLMediaElement_MozCaptureStream_FirefoxOS_2_2.pdf?raw=true

Comment on attachment 8549129 [details] [diff] [review] patch - Remove DestroyDecodedStream() from MediaDecoder::SetDormantIfNecessary() Review of attachment 8549129 [details] [diff] [review]: ----------------------------------------------------------------- Probably better to get Roc to check this, since it touches MediaStreams.

Comment on attachment 8549129 [details] [diff] [review] patch - Remove DestroyDecodedStream() from MediaDecoder::SetDormantIfNecessary() Approval Request Comment [Feature/regressing bug #]: MSE [User impact if declined]: Youtube playback can use more resources. Less consistent testing. [Describe test coverage new/current, TreeHerder]: Landed on m-c. [Risks and why]: Small, isolated change, but does affect non-MSE playback. [String/UUID change made/needed]: None.