1647112 - Make CanvasThreadHolder use a BackgroundTaskQueue

Status: RESOLVED FIXED

(Core :: Canvas: 2D, enhancement)

Description

[Jean-Yves Avenard [:jya]]

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

Comment 1

[Jean-Yves Avenard [:jya]]

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

Some headers cleanup.

Comment 2

[Jean-Yves Avenard [:jya]]

Attachment: Bug 1647112 - P2. Remove static method. 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

Comment 3

[Pulsebot]

Pushed by [e-mail]:
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

Comment 4

[Narcis Beleuzu [:NarcisB]]

Backed out for webgl-conf crashes

Backout link: https://hg.mozilla.org/integration/autoland/rev/ed44d4e62140475b5daba4e7db38808dfea95960
Log link: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=307988602&repo=autoland&lineNumber=3484

Comment 5

[Pulsebot]

Pushed by [e-mail]:
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

Comment 6

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/c3896c11842c
https://hg.mozilla.org/mozilla-central/rev/4f21c4c72174