Closed Bug 1314219 Opened 4 years ago Closed 4 years ago

Dormant shouldn't have observable effects on the media element

Categories

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

defect

Tracking

()

RESOLVED FIXED
mozilla52
Tracking Status
firefox52 --- fixed

People

(Reporter: jwwang, Assigned: jwwang)

References

Details

Attachments

(2 files)

For now, exiting dormant requires seeking which changes the next frame status and readyState of the media element.

Per bug 1311872 comment 16, we want entering/exiting dormant to be transparent so it won't cause regression of mochitests and players.
Assignee: nobody → jwwang
Blocks: 1286129
Depends on: 1311872
Priority: -- → P1
Attachment #8808907 - Flags: review?(cpearce)
Attachment #8808908 - Flags: review?(cpearce)
Comment on attachment 8808907 [details]
Bug 1314219. Part 1 - align next frame status updates with state changes of MDSM.

https://reviewboard.mozilla.org/r/91624/#review91814
Attachment #8808907 - Flags: review?(cpearce) → review+
Comment on attachment 8808908 [details]
Bug 1314219. Part 2 - remove unused watchables and fix some comments.

https://reviewboard.mozilla.org/r/91626/#review91816
Attachment #8808908 - Flags: review?(cpearce) → review+
Thanks!
Pushed by jwwang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/f82d06a8a42a
Part 1 - align next frame status updates with state changes of MDSM. r=cpearce
https://hg.mozilla.org/integration/autoland/rev/e39a5f086e68
Part 2 - remove unused watchables and fix some comments. r=cpearce
https://hg.mozilla.org/mozilla-central/rev/f82d06a8a42a
https://hg.mozilla.org/mozilla-central/rev/e39a5f086e68
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
Depends on: 1322087
You need to log in before you can comment on or make changes to this bug.