Closed
Bug 1747172
Opened 3 years ago
Closed 3 years ago
Make Pivot text functions use HyperTextAccessibleBase
Categories
(Core :: Disability Access APIs, task)
Tracking
()
RESOLVED
FIXED
99 Branch
| Tracking | Status | |
|---|---|---|
| firefox99 | --- | fixed |
People
(Reporter: Jamie, Assigned: Jamie)
References
(Blocks 1 open bug)
Details
(Whiteboard: [ctw-m1])
Attachments
(1 file)
|
(deleted),
text/x-phabricator-request
|
Details |
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.
|
Assignee | |
Comment 1•3 years ago
|
||
The XPCOM interface and AccVCChangeEvent still only support LocalAccessible.
These will need to be updated/refactored in the future.
|
||
Updated•3 years ago
|
Assignee: nobody → jteh
Status: NEW → ASSIGNED
Pushed by jteh@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8a3535e92f63
Support cached RemoteAccessible in Pivot text methods. r=eeejay
|
||
Comment 3•3 years ago
|
||
Backed out changeset 8a3535e92f63 (Bug 1747172) for causing build bustages on AccessibleWrap.cpp.
Backout link
Push with failures
Failure Log
Flags: needinfo?(jteh)
|
|
Assignee | |
Updated•3 years ago
|
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
|
||
Comment 5•3 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
status-firefox99:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 99 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•