Closed
Bug 1647112
Opened 5 years ago
Closed 5 years ago
Make CanvasThreadHolder use a BackgroundTaskQueue
Categories
(Core :: Graphics: Canvas2D, enhancement)
Core
Graphics: Canvas2D
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.
Assignee | ||
Comment 1•5 years ago
|
||
Some headers cleanup.
Updated•5 years ago
|
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
Assignee | ||
Comment 2•5 years ago
|
||
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
Comment 4•5 years ago
|
||
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
Flags: needinfo?(jyavenard)
Assignee | ||
Updated•5 years ago
|
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
Comment 6•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/c3896c11842c
https://hg.mozilla.org/mozilla-central/rev/4f21c4c72174
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox80:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla80
You need to log in
before you can comment on or make changes to this bug.
Description
•