Remove MDSM::DispatchMinimizePrerollUntilPlaybackStarts

RESOLVED FIXED in Firefox 53

Status

()

P5
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: jwwang, Assigned: jwwang)

Tracking

unspecified
mozilla53
Points:
---

Firefox Tracking Flags

(firefox53 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
http://searchfox.org/mozilla-central/rev/82ebc0c5ea45477ef4a4bfb3bbff35069bd18cee/dom/media/MediaDecoder.cpp#635

MediaDecoder::SetStateMachineParameters() is the only caller. We can let MDSM read MediaDecoder::mMinimizePreroll during initialization to avoid an additional task dispatch.
(Assignee)

Updated

2 years ago
Assignee: nobody → jwwang
Blocks: 1324999
Priority: -- → P5
Comment hidden (mozreview-request)
(Assignee)

Updated

2 years ago
Attachment #8824853 - Flags: review?(kaku)

Comment 2

2 years ago
mozreview-review
Comment on attachment 8824853 [details]
Bug 1328836 - Remove MDSM::DispatchMinimizePrerollUntilPlaybackStarts.

https://reviewboard.mozilla.org/r/103150/#review103718
Attachment #8824853 - Flags: review?(kaku) → review+
(Assignee)

Comment 3

2 years ago
Thanks!

Comment 4

2 years ago
Pushed by jwwang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1e65a894efcc
Remove MDSM::DispatchMinimizePrerollUntilPlaybackStarts. r=kaku

Comment 5

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/1e65a894efcc
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox53: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.