166322 - Accessibility: better support for XUL tree

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Kyle Yuan]

miscellaneous nsXULTreeAccessible problems will be fixed there.

Comment 1

[Kyle Yuan]

Attachment: patch

this patch fixed:
1) new algorithm to generate the accId for treeitem - use |this| pointer, that
is unique id;
2) fire focus event instead of selection-changed, because gnopernicus doesn't
care about selection-changed yet;
3) need to call EnsureColumns() before use mColumns in
nsTreeBodyFrame::GetKeyColumnIndex()

seeking r=

Comment 2

[Aaron Leventhal]

Comment on attachment 98081 [details] [diff] [review]
patch

r= aaronl

We should be are that the unique ID solution is not necessarily better with
this patch.

If the client app caches the entire accessible tree, and then tries to compare
the unique ID for a tree item that got focused, with the unique ID's in the
cached tree, the ID's will not match.

Comment 3

[Johnny Stenback (:jst)]

Comment on attachment 98081 [details] [diff] [review]
patch

sr=jst

Comment 4

[Kyle Yuan]

checked in!