1787283 - Cache NODE_CHILD_OF and NODE_PARENT_OF relations

Status: RESOLVED FIXED

(Core :: Disability Access APIs, task)

Description

[Eitan Isaacson [:eeejay]]

Marking as m3 because this is needed for Mac ctw.

Comment 1

[Morgan Reschenberg [:morgan]]

ETA: we also need the "parent" version of this

Comment 2

[Morgan Reschenberg [:morgan]]

:Jamie are these supposed to be reciprocal relations? It looks like they parent/child relationship gets mirrored for tree-related accs, but not for accs that hit this clause

Comment 3

[James Teh [:Jamie]]

As discussed on Zoom, they should probably be reciprocal strictly speaking. However, since this is now only used to deal with a very specific obscure case with legacy window emulation on Windows, it isn't really worth the implementation effort. We should just handle this special case in a similar way in DocAccessibleParent.

Comment 4

[James Teh [:Jamie]]

Expanding on comment 3, I mean that the NODE_CHILD_OF relation that we implement primarily for JAWS does not need a reciprocal NODE_PARENT_OF relation. For the JAWS case, we would implement this in a DocAccessibleParent::RelationByType overide, which would just return Parent() for NODE_CHILD_OF but only if IsTopLevel().

Comment 5

[Nathan LaPré]

Attachment: Bug 1787283: Provide NODE_CHILD_OF, NODE_PARENT_OF relations in remote acc, r?morgan

We'd like to provide NODE_CHILD_OF and NODE_PARENT_OF relations in the parent
process, relying on cached information, to avoid sync IPDL messages. This
revision implements most of the use cases for this relation for remote
accessibles, notably setting aside handling of MathML's <mroot> parent/child
relation. ARIA trees, treegrids, lists, and so on are handled in the remote
accessible largely identically to how they were handled in LocalAccessible.
Rather than define a new rule for walking the tree to find children, this
revision unifies the ItemIterator so it works on generic Accessibles, and uses
it in RemoteAccessibleBase the same way it's used in LocalAccessible. The
special case carve-out for MSAA clients now exists as a simpler IsTopLevel
check in DocAccessibleParent.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/62bd2f62aa5a
Provide NODE_CHILD_OF, NODE_PARENT_OF relations in remote acc, r=morgan,Jamie

Comment 7

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/62bd2f62aa5a