1747172 - Make Pivot text functions use HyperTextAccessibleBase

Status: RESOLVED FIXED

(Core :: Disability Access APIs, task)

Description

[James Teh [:Jamie]]

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.

Comment 1

[James Teh [:Jamie]]

Attachment: Bug 1747172: Support cached RemoteAccessible in Pivot text methods.

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

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8a3535e92f63
Support cached RemoteAccessible in Pivot text methods. r=eeejay

Comment 3

[Marian-Vasile Laza]

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

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e644785b83a6
Support cached RemoteAccessible in Pivot text methods. r=eeejay

Comment 5

[Cosmin Sabou [:CosminS]]

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