Closed Bug 1741792 Opened 3 years ago Closed 2 years ago

Cache the caret

Categories

(Core :: Disability Access APIs, task)

task

Tracking

()

RESOLVED FIXED
96 Branch
Tracking Status
firefox96 --- fixed

People

(Reporter: Jamie, Assigned: Jamie)

References

Details

(Whiteboard: [ctw-m0])

Attachments

(2 files)

We need to cache the caret Accessible and offset in the parent process cache. We can use caret events for this, since they include the offset. An open question is whether we cache the HyperText offset (as provided by the event) or transform it to a text leaf offset. I'd like to move everything towards text leaf, but we're not changing the event here and most clients will end up fetching a HyperText offset anyway, so this is not as clear as it might seem.

Whiteboard: [ctw-m0]
No longer blocks: 1730862
Depends on: 1730862

Caret events don't specify whether the caret is at the end of a line, but we need to cache this.
WE now include this info when we send a caret event to the parent process.

This includes support in both TextLeafPoint and HyperTextAccessibleBase.

Pushed by jteh@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/81c9a2ece881
part 1: Cache the caret in the parent process using caret events. r=eeejay
https://hg.mozilla.org/integration/autoland/rev/e171bed83754
part 2: Use the cached caret. r=eeejay
Regressions: 1743755
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 96 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: