Closed Bug 1911750 Opened 11 months ago Closed 10 months ago

layers.acceleration.force-enabled is enabled on Android by default

Categories

(Core :: Graphics, defect)

Unspecified
Android
defect

Tracking

()

RESOLVED FIXED
132 Branch
Tracking Status
firefox132 --- fixed

People

(Reporter: gregp, Assigned: gregp)

Details

Attachments

(1 file)

layers.acceleration.force-enabled has been 'temporarily' enabled by default on Android for about 12 years (bug 838603)

on Android, accidentally blacklisting OpenGL layers means a startup crash for everyone. Temporarily force-enable GL compositing. This is default-disabled deep within the bowels of the widgetry system.
Remove me when GL compositing isn't default disabled in widget/android.

https://searchfox.org/mozilla-central/rev/213d5278e59db7e6142e2b093d3fd0172f6392a4/modules/libpref/init/StaticPrefList.yaml#8382-8391

This seems like a hack that shouldn't be necessary these days? gfxAndroidPlatform::AccelerateLayersByDefault returns true, so it seems like we could switch the default value of this pref back to false and there would be no behavior change.

Without looking into this closely it seems reasonable to me. Jamie?

Flags: needinfo?(jnicol)
Severity: -- → S3

Yeah this seems reasonable. I tested setting it false and we still get acceleration. And then tested setting layers.acceleration.disabled to true, and we no longer get a startup crash, we instead get software webrender with GL compositing. So let's remove it

Flags: needinfo?(jnicol)
Assignee: nobody → gregp
Status: NEW → ASSIGNED
Pushed by gp3033@protonmail.com: https://hg.mozilla.org/integration/autoland/rev/9ae21b991a94 Disable layers.acceleration.force-enabled on Android r=jnicol
Status: ASSIGNED → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → 132 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: