Closed Bug 1747172 Opened 2 years ago Closed 2 years ago

Make Pivot text functions use HyperTextAccessibleBase

Categories

(Core :: Disability Access APIs, task)

ARM64
Android
task

Tracking

()

RESOLVED FIXED
99 Branch
Tracking Status
firefox99 --- fixed

People

(Reporter: Jamie, Assigned: Jamie)

References

Details

(Whiteboard: [ctw-m1])

Attachments

(1 file)

Android uses Pivot::Next/PrevText. These currently only support local HyperTextAccessibles.

Ideally, we'd convert Android to use TextLeafPoint/Range directly, but I think this is going to be more complicated and it means we can't leverage our existing tests. Therefore, I think it makes more sense initially to make it use HyperTextAccessibleBase.

Depends on: 1754360

The XPCOM interface and AccVCChangeEvent still only support LocalAccessible.
These will need to be updated/refactored in the future.

Assignee: nobody → jteh
Status: NEW → ASSIGNED
Blocks: 1754905
Pushed by jteh@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8a3535e92f63
Support cached RemoteAccessible in Pivot text methods. r=eeejay

Backed out changeset 8a3535e92f63 (Bug 1747172) for causing build bustages on AccessibleWrap.cpp.
Backout link
Push with failures
Failure Log

Flags: needinfo?(jteh)
Flags: needinfo?(jteh)
Pushed by jteh@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e644785b83a6
Support cached RemoteAccessible in Pivot text methods. r=eeejay
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 99 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: