Closed Bug 1370130 Opened 3 years ago Closed 3 years ago

MDSM shouldn't enter dormant when the dormant timeout is negative

Categories

(Core :: Audio/Video: Playback, enhancement, P5)

enhancement

Tracking

()

RESOLVED FIXED
mozilla55
Tracking Status
firefox55 --- fixed

People

(Reporter: jwwang, Assigned: jwwang)

Details

Attachments

(1 file)

http://searchfox.org/mozilla-central/rev/507743376d1ba753d14ab6b9305b7c6358570be8/dom/media/MediaDecoderStateMachine.cpp#897-905

We should check |timeout| before first so we can disable dormant using prefs even when preload is 'metadata'.
Assignee: nobody → jwwang
Priority: -- → P5
Attachment #8874281 - Flags: review?(kaku)
Comment on attachment 8874281 [details]
Bug 1370130 - dormant timeout should take precedence over preload='metadata'.

https://reviewboard.mozilla.org/r/145648/#review149618
Attachment #8874281 - Flags: review?(kaku) → review+
Thanks!
Pushed by jwwang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6e9e40efea39
dormant timeout should take precedence over preload='metadata'. r=kaku
https://hg.mozilla.org/mozilla-central/rev/6e9e40efea39
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.