Enable accessibility caching by default

RESOLVED FIXED in Firefox 64

Status

defect
P1
normal
RESOLVED FIXED
9 months ago
8 months ago

People

(Reporter: eeejay, Assigned: eeejay)

Tracking

unspecified
mozilla65
Dependency tree / graph

Firefox Tracking Flags

(geckoview64 fixed, firefox64 fixed, firefox65 fixed)

Details

Attachments

(1 attachment)

...disable full tree a11y by default.
Blocks: 1499163, 1502366
This would mean we would be using the cache exclusively unless otherwise
set. For now, the autofill tests are still set to use full tree a11y.
This is mostly because of kinks in how the test starts, as well as the
inputType that is not stored in the viewport cache. We should have a
followup bug to deal with that.
Depends on: 1507026
Based on bug 1499163 comment 26 and 27, it sounds like we'll want to uplift this fix to Fennec 64 Beta.
Priority: -- → P1
Pushed by eisaacson@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2814e983657b
Disable Android full tree accessibility by default. r=snorp
https://hg.mozilla.org/mozilla-central/rev/2814e983657b
Status: NEW → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 65
Comment on attachment 9024595 [details]
Bug 1506709 - Disable Android full tree accessibility by default. r?snorp!

[Beta/Release Uplift Approval Request]

Feature/Bug causing the regression: Bug 1479037

User impact if declined: Users may potentially experience slowdowns if they have accessibility enabled (screen reader, password manager, etc)

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: Bug 1501496, Bug 1479039, Bug 1507026, Bug 1506709, Bug 1502187, Bug 1507026, Bug 1505515

Risk to taking this patch: Medium

Why is the change risky/not risky? (and alternatives if risky): The risk here is moderate but notable.

String changes made/needed:
Attachment #9024595 - Flags: approval-mozilla-beta?
Comment on attachment 9024595 [details]
Bug 1506709 - Disable Android full tree accessibility by default. r?snorp!

android a11y perf fix, approved for 64.0b12
Attachment #9024595 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
status-geckoview64=fixed
Product: Firefox for Android → GeckoView
Target Milestone: Firefox 65 → mozilla65
You need to log in before you can comment on or make changes to this bug.