Incorrectly rendered pie chart on Wikipedia
Categories
(Core :: Graphics, defect)
Tracking
()
People
(Reporter: 1justinpeter, Assigned: jnicol)
References
(Regression, )
Details
(Keywords: regression)
Attachments
(1 file)
48 bytes,
text/x-phabricator-request
|
pascalc
:
approval-mozilla-release+
dmeehan
:
approval-mozilla-esr115+
|
Details | Review |
(Not sure if this is the right component or not; apologies and please move it if it isn't)
Reported on Wikpedia here; some (rather janky, from the sounds of it) HTML pie charts are not displaying properly.
This appears to happen with Firefox for Android on some devices but not others. The one person who has confirmed this occurring is on a device with a Mali-G77.
Updated•1 year ago
|
Updated•1 year ago
|
Comment 1•1 year ago
|
||
I can reproduce artifacts on desktop since at least 2020.
Reporter | ||
Comment 2•1 year ago
|
||
Two quick comments:
- This appears to be somehow device/driver specific; I can't reproduce it on Windows or Android.
- This is not conic gradient - there's talk of replacing this current system with a conic gradent-based one, but this is not rendered that way.
Updated•1 year ago
|
Assignee | ||
Comment 3•1 year ago
|
||
Can reproduce on Pixel 7 (Mali G710)
Bug 1823411 makes sense
Assignee | ||
Comment 4•1 year ago
|
||
This was caused by reducing the precision of vColorLine
in cs_border_solid
to mediump
. It's unclear why this only occurs on a specific range of Mali GPUs. Also it does not occur with shader optimisation disabled, and we know that doesn't always handle precision correctly. So there could perhaps be more to investigate there.
But there's a strong argument that this should never have been made mediump
in the first place, as calculations regarding positions etc require the extra precision. So I think the best solution is simply to revert it to highp.
Assignee | ||
Comment 5•1 year ago
|
||
This varying's precision being reduced to mediump in bug bug 1823411
is causing rendering glitches on certain Mali devices. It should never
have been reduced in the first place, as calculations regarding
positions may require the extra precision. This patch therefore
reverts it to highp.
Pushed by jnicol@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/aee77156969b Make cs_border_solid's vColorLine highp. r=gfx-reviewers,bradwerth
Comment 7•1 year ago
|
||
bugherder |
Updated•1 year ago
|
Comment 8•1 year ago
|
||
Jamie, is this a good candidate for uplifting in beta and/or our planned dot release? Thanks
Assignee | ||
Comment 9•1 year ago
|
||
I think this landed before 119 branched to beta so no point uplifting there? but I think it's a good candidate for the planned dot release
Assignee | ||
Comment 10•1 year ago
|
||
Comment on attachment 9354595 [details]
Bug 1850815 - Make cs_border_solid's vColorLine highp. r?#gfx-reviewers
Beta/Release Uplift Approval Request
- User impact if declined: Incorrect rendering of certain elements on some devices with Mali GPUs (including Pixel 6 and 7)
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Reverts shader back to its previous code before the bug was introduced
- String changes made/needed:
- Is Android affected?: Yes
Comment 11•11 months ago
|
||
Comment on attachment 9354595 [details]
Bug 1850815 - Make cs_border_solid's vColorLine highp. r?#gfx-reviewers
Approved for our next 118 dot release, thanks.
Comment 12•11 months ago
|
||
uplift |
https://hg.mozilla.org/releases/mozilla-release/rev/6a10dcee7423
Comment 13•11 months ago
|
||
bugherder uplift |
Updated•11 months ago
|
Comment 14•11 months ago
|
||
Comment on attachment 9354595 [details]
Bug 1850815 - Make cs_border_solid's vColorLine highp. r?#gfx-reviewers
Approved for 115.4esr
Comment 15•11 months ago
|
||
uplift |
https://hg.mozilla.org/releases/mozilla-esr115/rev/c6a32b928430
Updated•11 months ago
|
Description
•