Closed Bug 1343732 Opened 3 years ago Closed 3 years ago

Label runnables in dom/media/mediasource/AutoTaskQueue.h

Categories

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

enhancement

Tracking

()

RESOLVED FIXED
mozilla54
Tracking Status
firefox54 --- fixed

People

(Reporter: jwwang, Assigned: jwwang)

References

Details

(Whiteboard: [QDL][TDC-MVP][MEDIA])

Attachments

(1 file)

http://searchfox.org/mozilla-central/rev/31b6089ce26fa76459642765115605d50a6c67b4/dom/media/mediasource/AutoTaskQueue.h#54

Since |taskqueue->BeginShutdown()| doesn't run scripts, we can use SystemGroup::Dispatch() to dispatch the task instead of the abstract main thread.

This also save one parameter to the constructor.
Assignee: nobody → jwwang
Blocks: 1341539
Priority: -- → P3
Comment on attachment 8842703 [details]
Bug 1343732 - use SystemGroup::Dispatch() in ~AutoTaskQueue().

https://reviewboard.mozilla.org/r/116462/#review118238
Attachment #8842703 - Flags: review?(jyavenard) → review+
Thanks!
Pushed by jwwang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/11dc8f6f7ed0
use SystemGroup::Dispatch() in ~AutoTaskQueue(). r=jya
https://hg.mozilla.org/mozilla-central/rev/11dc8f6f7ed0
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
Whiteboard: [QDL][TDC-MVP][MEDIA]
You need to log in before you can comment on or make changes to this bug.