Closed Bug 1647112 Opened 5 years ago Closed 5 years ago

Make CanvasThreadHolder use a BackgroundTaskQueue

Categories

(Core :: Graphics: Canvas2D, enhancement)

enhancement

Tracking

()

RESOLVED FIXED
mozilla80
Tracking Status
firefox80 --- fixed

People

(Reporter: jya, Assigned: jya)

References

Details

Attachments

(2 files)

Now that bug 1634846 allows the use of TaskQueue with IPC's MessageChannel we can use background taskqueue with canvas.

Attachment #9158429 - Attachment description: Bug 1647112 - Make the canvas thread use BackgroundTaskQueue. r?mattwoodrow → Bug 1647112 - P1. Make the canvas thread use BackgroundTaskQueue. r?mattwoodrow

Allows to remove the dance around the compositor thread just to perform the assertion.
The use of an in-line lambda avoid all ambiguity on which thread this is called.

Depends on D80597

Pushed by jyavenard@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/2d24ad813039 P1. Make the canvas thread use BackgroundTaskQueue. r=KrisWright https://hg.mozilla.org/integration/autoland/rev/4efaf32bc8f7 P2. Remove static method. r=mattwoodrow
Flags: needinfo?(jyavenard)
Pushed by jyavenard@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/c3896c11842c P1. Make the canvas thread use BackgroundTaskQueue. r=KrisWright,jgilbert,andi https://hg.mozilla.org/integration/autoland/rev/4f21c4c72174 P2. Remove static method. r=mattwoodrow
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla80
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: