Closed Bug 1836726 Opened 11 months ago Closed 11 months ago

Subpixel AA cannot be force-disabled to work around performance issues

Categories

(Core :: Graphics: WebRender, enhancement, P2)

Firefox 116
enhancement

Tracking

()

RESOLVED FIXED
116 Branch
Tracking Status
firefox116 --- fixed

People

(Reporter: ke5trel, Assigned: gw)

References

(Blocks 1 open bug)

Details

(Keywords: perf)

Attachments

(1 file)

Subpixel AA text rendering significantly reduces scrolling performance on some sites like reddit. This was especially noticeable for me on Ubuntu 23.04 with AMD RS880 (R600) when testing builds after Bug 1600472 (Disable allowing sacrificing subpixel anti-aliasing for small screens) which killed scrolling smoothness, later partly addressed by Bug 1635610 (Enable per scroll root picture cache slices by default) but it is still not as smooth as before.

It's not possible to do a comparison on the latest Nightly version since there is no way to turn subpixel AA off in WR, only force-enable with gfx.webrender.quality.force-subpixel-aa-where-possible = true. gfx.text.disable-aa = true does not use grayscale and does not seem to affect WR, giving no performance benefit. Changing the system setting /org/gnome/desktop/interface/font-antialiasing to grayscale or none has no effect and fontconfig settings also don't work (Bug 1726312).

Subpixel AA is expensive and unnecessary in many cases, it is not needed for high resolution screens (Bug 1698701), it is disabled on Android (Bug 1527807) and WR often sacrifices it for performance anyway. Exposing an about:config setting to let users disable it for themselves would help identify and work around associated performance issues.

Glenn, could this be a quick-fix for Nightly, or are there deeper concerns with providing this?

Severity: -- → S3
Type: defect → enhancement
Flags: needinfo?(gwatson)
Priority: -- → P2

It's likely to be something fairly simple, I'll take a look this week.

Assignee: nobody → gwatson
Flags: needinfo?(gwatson)

Previously, this was hard coded to off for android, on for everything
else. With this change, it's controlled by a preference which
defaults to off for android, and on for everything else. This allows
users to disable subpixel AA on desktop for performance reasons.

This is a straightforward change, so I've put a patch up to add this functionality. Thanks for the detailed report.

Pushed by gwatson@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a46936121982
Allow disabling subpixel AA in WR via preference r=gfx-reviewers,lsalzman
Status: NEW → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → 116 Branch
No longer blocks: 1828098
See Also: → 1828098
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: