Closed Bug 1121658 Opened 5 years ago Closed 5 years ago

Remove DestroyDecodedStream() from MediaDecoder::SetDormantIfNecessary()

Categories

(Core :: Audio/Video, defect, P1)

ARM
Gonk (Firefox OS)
defect

Tracking

()

RESOLVED FIXED
mozilla38
Tracking Status
firefox36 --- fixed
firefox37 --- fixed
firefox38 --- fixed

People

(Reporter: sotaro, Assigned: sotaro)

References

Details

Attachments

(1 file)

+++ 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.
Assignee: nobody → sotaro.ikeda.g
Attachment #8549129 - Flags: review?(cpearce)
Blocks: 1050031
No longer blocks: 1121064
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.
Attachment #8549129 - Flags: review?(cpearce) → review?(roc)
https://hg.mozilla.org/mozilla-central/rev/56824531bfec
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
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.
Attachment #8549129 - Flags: approval-mozilla-beta?
Attachment #8549129 - Flags: approval-mozilla-aurora?
Attachment #8549129 - Flags: approval-mozilla-beta?
Attachment #8549129 - Flags: approval-mozilla-beta+
Attachment #8549129 - Flags: approval-mozilla-aurora?
Attachment #8549129 - Flags: approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.