Closed
Bug 1454980
Opened 7 years ago
Closed 6 years ago
Let POMTP tiling with skia on windows ride the trains
Categories
(Core :: Graphics: Layers, enhancement, P3)
Core
Graphics: Layers
Tracking
()
RESOLVED
FIXED
mozilla62
People
(Reporter: rhunt, Assigned: rhunt)
References
Details
(Keywords: feature, Whiteboard: [gfx-noted])
Attachments
(1 file)
|
735 bytes,
patch
|
davidb
:
review+
|
Details | Diff | Splinter Review |
This is the bug to track letting bug 1454978 ride the trains.
Manual testing passed.
|
Assignee | |
Comment 2•6 years ago
|
||
With all outstanding bugs resolved and manual testing passed, I think we are good to let this ride the trains.
Attachment #8981481 -
Flags: review?(dbolter)
|
||
Comment 3•6 years ago
|
||
Comment on attachment 8981481 [details] [diff] [review]
pomtp-trains.patch
Review of attachment 8981481 [details] [diff] [review]:
-----------------------------------------------------------------
\o/
Attachment #8981481 -
Flags: review?(dbolter) → review+
Pushed by rhunt@eqrion.net:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9c587c9fb7de
Let tiled parallel skia on unaccelerated windows ride the trains. r=davidb
|
||
Comment 5•6 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox62:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
|
||
Updated•6 years ago
|
tracking-firefox62:
--- → ?
|
||
Comment 6•6 years ago
|
||
This already landed, but we can track it to make sure to catch if it reopens.
|
|
||
Comment 7•6 years ago
|
||
Is this something you'd like in the release notes for 62?
Flags: needinfo?(dbolter)
|
|
||
Comment 8•6 years ago
|
||
Yes and thanks for catching Liz. Probably best to get a one-liner straight from Ryan.
Flags: needinfo?(dbolter) → needinfo?(rhunt)
|
|
Assignee | |
Comment 9•6 years ago
|
||
I looked at some past release notes for OMTP and wrote one. Feel free to workshop it.
"Improved graphics rendering using Parallel-Off-Main-Thread Painting (P-OMTP) for Windows users without accelerated hardware (P-OMTP for OSX was released in Firefox 60)"
Note: I looked back at the release notes for Firefox 60 and noticed that P-OMTP being enabled on OSX was not mentioned there. It must have gotten lost somehow. It was originally enabled in bug 1430793.
Flags: needinfo?(rhunt)
|
|
||
Updated•6 years ago
|
relnote-firefox:
--- → ?
Added to Firefox 62 release notes.
You need to log in
before you can comment on or make changes to this bug.
Description
•