1657653 - Calling certain text operations in mac hangs firefox

Status: VERIFIED FIXED

(Core :: Disability Access APIs, defect, P1)

Description

[Eitan Isaacson [:eeejay]]

The current implementation for getting a string for a range requires us to traverse the text tree in the parent process and make a handful of synchronous calls into content for each node. The larger the subtree, the more calls. This means that if a user selects a very large chunk of text with VoiceOver running, or OSX's speak selection feature - Firefox will hang.

I think we need to back out this API until a more efficient implementation is available.

Comment 1

[Eitan Isaacson [:eeejay]]

Attachment: Bug 1657653 - Hide text marker range attributes that hammer IPC. r=morgan

Comment 2

[Eitan Isaacson [:eeejay]]

Comment on attachment 9169466 [details]
Bug 1657653 - Hide text marker range attributes that hammer IPC. r=morgan

Beta/Release Uplift Approval Request

• User impact if declined: Serious firefox slow downs
• 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): This has landed in central. It mistakenly got associated with another bug:
  https://hg.mozilla.org/mozilla-central/rev/e43038a3c22a8aaf90d266048bf4bc4272111365
• String changes made/needed:

Comment 3

[Julien Cristau [:jcristau]]

https://hg.mozilla.org/mozilla-central/rev/e43038a3c22a8aaf90d266048bf4bc4272111365

Comment 4

[Julien Cristau [:jcristau]]

Comment on attachment 9169466 [details]
Bug 1657653 - Hide text marker range attributes that hammer IPC. r=morgan

macos a11y fix, approved for 80.0b8

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/aff6ff228795

Comment 6

[Catalin Sasca, Desktop Test Engineering [:csasca]]

I successfully reproduced the issue on Firefox 81.0a1 (2020-08-06) under macOS 11 by selecting all the text from this page and activate the speak selection feature.

The hangs are no longer reproducible on Firefox Nightly 81.0a1 (2020-08-16), but I couldn't use the text selection feature on 80.0b8. It simply won't activate by using the key combinations.

Any idea what could have happened here Eitan?

Comment 7

[Eitan Isaacson [:eeejay]]

(In reply to Catalin Sasca, QA [:csasca] from comment #6)

I successfully reproduced the issue on Firefox 81.0a1 (2020-08-06) under macOS 11 by selecting all the text from this page and activate the speak selection feature.

The hangs are no longer reproducible on Firefox Nightly 81.0a1 (2020-08-16), but I couldn't use the text selection feature on 80.0b8. It simply won't activate by using the key combinations.

Any idea what could have happened here Eitan?

The patch in this bug temporarily deactivated the speak selection feature until we could figure out a performant way to do it. It will remain disabled in 80, but I recently landed a patch that makes it fast again in 81, and re-exposed the API in bug 1659215. So now this is fixed and should not be reproducible in nightly.

Comment 8

[Catalin Sasca, Desktop Test Engineering [:csasca]]

Marking nightly as fixed, as it was verified on 81.0a1 (2020-08-16), see Comment 6.

Comment 9

[Cristian Fogel [:cfogel]]

Verified as well with Firefox 80.0 on macOS 10.15.