Closed Bug 1764249 Opened 3 years ago Closed 3 years ago

With the cache enabled, document incorrectly reports text selection when there is a caret

Categories

(Core :: Disability Access APIs, defect)

defect

Tracking

()

VERIFIED FIXED
101 Branch
Tracking Status
firefox-esr91 --- unaffected
firefox99 --- wontfix
firefox100 --- wontfix
firefox101 --- verified

People

(Reporter: Jamie, Assigned: Jamie)

References

(Blocks 1 open bug, Regression)

Details

(Keywords: regression, Whiteboard: [ctw-m2])

Attachments

(1 file)

STR:

  1. Set the pref accessibility.cache.enabled to true.
  2. Open this test case:
    data:text/html,<div role="group"><div contenteditable>test</div></div>
  3. Focus the editable area.
  4. Use the arrow keys to move the caret.
  5. Query the selection count on the document accessible.
    • Expected: 0
    • Actual: 1

This behaves correctly when the cache is disabled. When it's enabled, a collapsed selection in a descendant gets reported on the DocAccessible. I don't know why yet.

Whiteboard: [ctw-m2]

Set release status flags based on info from the regressing bug 1741793

Has Regression Range: --- → yes

There's no point cropping ranges when called on the document, but we still want to remove collapsed ranges.
The path to hell is paved with premature optimisation. :(

Pushed by jteh@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/bbf84e358f0b Make HyperTextAccessibleBase::CroppedSelectionRanges remove collapsed ranges when called on a document. r=morgan
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 101 Branch
Flags: qe-verify+

Reproducible on Firefox 100.0(20220428192727) on macOS 11. Confirmed as fixed on Firefox 101.0b5(20220510144626) and Nightly 102.0a1(20220512094957) on macOS 11, Win10 64 bits and Ubuntu 20.04.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: