Closed Bug 1213897 Opened 6 years ago Closed 6 years ago

Extract DelayedScheduler out of MDSM to a common class for easier reuse by other classes.

Categories

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

defect

Tracking

()

RESOLVED FIXED
mozilla44
Tracking Status
firefox44 --- fixed

People

(Reporter: kikuo, Unassigned)

References

Details

Attachments

(1 file, 1 obsolete file)

According to the last issue in Bug 1194918 Comment 32, I think it's good to extract DelayedScheduler out of MDSM and make it more general for easier use by other classes.
Bug 1213897 - Extract DelayedScheduler out of MDSM to a common class. r?jwwang
Attachment #8672664 - Flags: review?(jwwang)
Blocks: 1194918
Comment on attachment 8672664 [details]
MozReview Request: Bug 1213897 - Extract DelayedScheduler out of MDSM to a common class. r?jwwang

https://reviewboard.mozilla.org/r/21761/#review19575

nice!
Attachment #8672664 - Flags: review?(jwwang) → review+
Keywords: checkin-needed
Hi Kilik, this failed to apply to mozilla-inbound

applying 6d94aba4bdb6
patching file dom/media/MediaDecoderStateMachine.cpp
Hunk #2 FAILED at 2855
1 out of 2 hunks FAILED -- saving rejects to file dom/media/MediaDecoderStateMachine.cpp.rej

could you take a look, thanks!
Flags: needinfo?(kikuo)
Keywords: checkin-needed
Rebase to trunk and carry r+ from Comment 2.
Should be applied without problem now : )

Thanks Carsten.
Flags: needinfo?(kikuo)
Attachment #8672954 - Flags: review+
Comment on attachment 8672664 [details]
MozReview Request: Bug 1213897 - Extract DelayedScheduler out of MDSM to a common class. r?jwwang

Avoid confusion
Attachment #8672664 - Attachment is obsolete: true
Keywords: checkin-needed
Priority: -- → P2
https://hg.mozilla.org/mozilla-central/rev/f9555c07f0e8
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
You need to log in before you can comment on or make changes to this bug.