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

Status: RESOLVED FIXED

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

Description

[Kilik Kuo [:kikuo] (inactive)]

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.

Comment 1

[Kilik Kuo [:kikuo] (inactive)]

Attachment: MozReview Request: Bug 1213897 - Extract DelayedScheduler out of MDSM to a common class. r?jwwang

Bug 1213897 - Extract DelayedScheduler out of MDSM to a common class. r?jwwang

Comment 2

[JW Wang [:jwwang] [:jw_wang]]

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!

Comment 3

[Kilik Kuo [:kikuo] (inactive)]

try run,
https://treeherder.mozilla.org/#/jobs?repo=try&revision=94c8219f31f6

Comment 4

[Carsten Book [:Tomcat]]

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!

Comment 5

[Kilik Kuo [:kikuo] (inactive)]

Attachment: Extract DelayedScheduler Out Of MDSM_rebase_to_267317

Rebase to trunk and carry r+ from Comment 2.
Should be applied without problem now : )

Thanks Carsten.

Comment 6

[Kilik Kuo [:kikuo] (inactive)]

Comment on attachment 8672664 [details]
MozReview Request: Bug 1213897 - Extract DelayedScheduler out of MDSM to a common class. r?jwwang

Avoid confusion

Comment 7

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/f9555c07f0e8

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/f9555c07f0e8