Blurred SVG - Background-image regression after 75.0b9
Categories
(Core :: Graphics: WebRender, defect)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr68 | --- | unaffected |
firefox76 | --- | wontfix |
firefox77 | --- | wontfix |
firefox78 | --- | fixed |
People
(Reporter: loic, Assigned: tnikkel)
References
(Regression, )
Details
(Keywords: regression)
Attachments
(7 files, 1 obsolete file)
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:77.0) Gecko/20100101 Firefox/77.0
Steps to reproduce:
Watch: https://codepen.io/MediaComem/pen/jOPveBv
The P letter is blurred on the right. This issue only happen since version 76.0 (last working version is 75.0b9)
Comment 1•5 years ago
|
||
Bugbug thinks this bug should belong to this component, but please revert this change in case of error.
Comment 2•5 years ago
|
||
Looks fine to me but I'm on a Mac. Perhaps you could find a regression range by using https://mozilla.github.io/mozregression/
Updated•5 years ago
|
Comment 3•5 years ago
|
||
Because this bug's Severity is normal
and has not been changed, and this bug's priority is --
(none,) indicating it has has not been previously triaged, the bug's Severity is being updated to --
(default, untriaged.)
I've run the tool, it crash after "autoland build: c030dc56" (I've tried 3 times and filled a bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1637164). But in the end, it seems that the bisection is done. Seems related to: https://phabricator.services.mozilla.com/D69167
Updated•5 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Updated•5 years ago
|
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 8•4 years ago
|
||
This is a reduced testcase.
Actually, this testcase shows a bug existed even before bug 1626259. This testcase is blurry even before bug 1626259 (whereas the original testcase looked pretty crisp before bug 1626259). It gets even more blurry after bug 1626259.
I should be able to make this testcase crisp, thus fixing the recent regression and fixing the pre-existing bug (it seemed to exist for quite a while before).
Assignee | ||
Comment 9•4 years ago
|
||
transform_style is set to frame->Extend3DContext, but ChooseScale uses frame->Combines3DTransformWithAncestors()
Assignee | ||
Comment 10•4 years ago
|
||
In StackingContextHelper::StackingContextHelper we want to handle the case of "no passed in transform" differently from "passed in transform but it cannot be drawn 2d". This is a little tricky because ChooseScale always has a transform passed in: in the non-wr case it is a scaling matrix by the parent scale. We could call ChooseScale if canDraw2D is false and get the same value for mScale but we also need to keep mInheritedTransform in sync.
This issue arose before: https://bugzilla.mozilla.org/show_bug.cgi?id=1449958#c3
And we want to be careful not to regress that bug, that bug was concerning content without a transform, so we should be good.
The testcase has a parent element with rotateY(-80deg) and a child element with rotateY(80deg), the combined transform is flat with no scaling even though each transform individually would have a very small x scale. There's no way to choose a good scale by looking at each transform individually, and since we only currently store transforms as 2d matrices we can't look at the full combined transform in order to choose a scale. Thus we must use a unit scale, like ChooseScale does.
Depends on D75288
Assignee | ||
Comment 11•4 years ago
|
||
The existing test compares a 3d scene to a "hand drawn" version of the scene using 2d primitives and was never expected to match perfectly, but instead just provide something to compare against better than saying it's not a blank page.
The new test looks visually identical, the fuzz is around the edges of the text.
Depends on D75289
Updated•4 years ago
|
Comment 12•4 years ago
|
||
Assignee | ||
Comment 13•4 years ago
|
||
Depends on D75636
Comment 14•4 years ago
|
||
Comment 15•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/14b1591d2a17
https://hg.mozilla.org/mozilla-central/rev/80fd72570c40
https://hg.mozilla.org/mozilla-central/rev/58f80ec5f2af
Comment 16•4 years ago
|
||
bugherder |
Description
•