1733514 - Add basic testing for text caching

Status: RESOLVED FIXED

(Core :: Disability Access APIs, task)

Description

[James Teh [:Jamie]]

ADD XPCOM support for the HyperTextAccessibleBase methods implemented in bug 1730088. Add some basic browser tests which utilise this to test cached text and word/line start offsets.

Comment 1

[James Teh [:Jamie]]

Attachment: Bug 1733514 part 1: Make HyperTextAccessibleBase::CharacterCount virtual and override it in RemoteAccessible.

When the cache is disabled, we need a different implementation of RemoteAccessible::CharacterCount for non-Windows.
This already existed, but it hid the base implementation rather than overriding it.
On Windows, we don't support this for RemoteAccessible at all unless the cache is enabled, so there's no change there.

Comment 2

[James Teh [:Jamie]]

Attachment: Bug 1733514 part 2: Rename RemoteAccessible::GetTextAtOffset to TextAtOffset and have it override HyperTextAccessibleBase.

Comment 3

[James Teh [:Jamie]]

Attachment: Bug 1733514 part 3: Rename xpcAccessibleHyperText::Intl to IntlLocal.

This is to make way for a new Intl which returns HyperTextAccessibleBase.

Comment 4

[James Teh [:Jamie]]

Attachment: Bug 1733514 part 4: Add xpcAccessibleHyperText::Intl and use it for methods which are supported by HyperTextAccessibleBase.

Comment 5

[James Teh [:Jamie]]

Attachment: Bug 1733514 part 5: Add initial tests for a11y text caching.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/29a1819168be
part 1: Make HyperTextAccessibleBase::CharacterCount virtual and override it in RemoteAccessible. r=eeejay
https://hg.mozilla.org/integration/autoland/rev/4030c1053bc8
part 2: Rename RemoteAccessible::GetTextAtOffset to TextAtOffset and have it override HyperTextAccessibleBase. r=eeejay
https://hg.mozilla.org/integration/autoland/rev/f487ae5b6da2
part 3: Rename xpcAccessibleHyperText::Intl to IntlLocal. r=eeejay
https://hg.mozilla.org/integration/autoland/rev/bee58fee1d01
part 4: Add xpcAccessibleHyperText::Intl and use it for methods which are supported by HyperTextAccessibleBase. r=eeejay
https://hg.mozilla.org/integration/autoland/rev/516a599e3187
part 5: Add initial tests for a11y text caching. r=eeejay

Comment 7

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/29a1819168be
https://hg.mozilla.org/mozilla-central/rev/4030c1053bc8
https://hg.mozilla.org/mozilla-central/rev/f487ae5b6da2
https://hg.mozilla.org/mozilla-central/rev/bee58fee1d01
https://hg.mozilla.org/mozilla-central/rev/516a599e3187