fission oop iframes lose cleartype on Windows
Categories
(Core :: Web Painting, defect, P3)
Tracking
()
Fission Milestone | MVP |
Tracking | Status | |
---|---|---|
firefox-esr68 | --- | unaffected |
firefox76 | --- | disabled |
firefox77 | --- | disabled |
firefox78 | --- | disabled |
People
(Reporter: tnikkel, Unassigned)
References
(Blocks 1 open bug)
Details
We seem to lose cleartype rendering in fission oop iframes on Windows, at least while running reftests in automation.
Example: https://treeherder.mozilla.org/#/jobs?repo=try&revision=dd5f6e44f6f56dfd113866025f751f0b9f28d840
(that is a try push with my work to make the reftest harness work better with fission, the same failure mode happens without my patches but you also get a failure mode where the test or reference iframe isn't drawn at all https://treeherder.mozilla.org/#/jobs?repo=try&revision=a83daddd6656ccde123884299e8c821dbd2dc90a for examples)
Updated•5 years ago
|
Reporter | ||
Comment 1•5 years ago
|
||
Matt informed me of the following: since the iframe doesn't have a bg color and the win7 machines are using basic compositor losing subpixel AA is expected there.
The win10 failure is different, they should have advanced layers and hence component alpha so they should still get subpixel AA. He suggests it might be a FrameLayerBuilder bug preventing it from getting component alpha.
Comment 2•5 years ago
|
||
Tentatively moving all bugs whose summaries mention "Fission" (or other Fission-related keywords) but are not assigned to a Fission Milestone to the "?" triage milestone.
This will generate a lot of bugmail, so you can filter your bugmail for the following UUID and delete them en masse:
0ee3c76a-bc79-4eb2-8d12-05dc0b68e732
Comment 4•5 years ago
|
||
According to tnikkel, this bug causes reftest fission tests to be orange.
Updated•5 years ago
|
Updated•5 years ago
|
Comment 5•5 years ago
|
||
This feels like it belongs in Web Painting because of the background color issue. Please feel free to push back if not.
Comment 7•5 years ago
|
||
Here is a recent try run with fission reftests: https://treeherder.mozilla.org/#/jobs?repo=try&revision=14cae9c8e8e455c30127fb8819cd41bb4fd88d3d
The four WebRender reftest failures do look like it could be caused by subpixel AA not working properly.
Comment 8•4 years ago
|
||
Matt, you're the triage owner for the Web Painting component. Do these Windows reftest failures need to block shipping Fission MVP? If so, do they also need to block enabling Fission in Nightly?
The test failures from Miko's try run in comment 7 look pretty minor in the reftest analyzer, but I don't know if these failures might hint at bigger problems with Fission rendering.
Comment 9•4 years ago
|
||
I don't think we need to block MVP/Nightly on these.
The issue is likely that we have OOP-iframes without a background color, that would normally have an opaque background from the outer doc.
That's not going to be a super common case, so I don't think it needs to block (except maybe Release?).
That said, it seems like WR should be able to render this fine, so we should try investigate at some point.
Comment 10•4 years ago
|
||
(In reply to Matt Woodrow (:mattwoodrow) from comment #9)
I don't think we need to block MVP/Nightly on these.
...That's not going to be a super common case, so I don't think it needs to block (except maybe Release?).
In that case, I'll track this bug for Fission M7 Beta as a reminder to re-evalutate these tests before we ride the trains.
Comment 11•4 years ago
|
||
Matt, can you please find an owner for this for Fission M7?
Comment 12•4 years ago
|
||
I don't think anything has changed since my comment 9 here, unless you're aware of pages where this causes an issue?
Comment 13•4 years ago
|
||
Given that we're planning to require WebRender for Fission, is this an issue anymore (based on comment 9)?
Moving bug to MVP based on comment 9
Comment 14•4 years ago
|
||
I'm going to close this for now, since the code in question definitely won't be running with WebRender.
If we find specific examples of regressions with WebRender enabled, then we can file a new bug for those.
Description
•