Don't do blocking IPC calls on each accessible event

RESOLVED FIXED in Firefox 64

Status

()

defect
RESOLVED FIXED
10 months ago
9 months ago

People

(Reporter: eeejay, Assigned: eeejay)

Tracking

(Blocks 1 bug)

unspecified
mozilla65
Unspecified
Android
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(geckoview64 fixed, firefox63 wontfix, firefox64 fixed, firefox65 fixed)

Details

(Whiteboard: [geckoview:p1])

Attachments

(1 attachment)

We currently send some info about the accessible with the event when dispatching to platform. This means we do a couple of synchronous IPC calls, specifically to get the accessible's name to populate the event's text field.

I checked, and Chrome doesn't do that anymore. TalkBack relies on the text on the actual accessible node it retrieves from the event. If we remove that text as well, it will be a couple of less synchronous calls we make.
Pushed by eisaacson@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6ff7a3acb857
Don't send accessible GeckoBundle in sendEvent, class names is enough. r=Jamie
https://hg.mozilla.org/mozilla-central/rev/6ff7a3acb857
Status: NEW → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
Depends on: 1505515
Comment on attachment 9019787 [details]
Bug 1501496 - Don't send accessible GeckoBundle in sendEvent, class names is enough. r?Jamie!

[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 1502187, Bug 1479039, Bug 1507026, Bug 1506709

Risk to taking this patch: Low

Why is the change risky/not risky? (and alternatives if risky): It simplifies accessibility events, so things should be better.

String changes made/needed:
Attachment #9019787 - Flags: approval-mozilla-beta?
64=affected because Eitan says this fix should be uplifted to 64 Beta.
[geckoview:p1] because this bug blocks Focus 8.0.
Whiteboard: [geckoview:p1]
Note that this patch seems to fix an OOM in 64 found in bug 1508641.
Blocks: 1508641
Comment on attachment 9019787 [details]
Bug 1501496 - Don't send accessible GeckoBundle in sendEvent, class names is enough. r?Jamie!

approved for 64.0b12
Attachment #9019787 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
status-geckoview64=fixed
You need to log in before you can comment on or make changes to this bug.