In the current code there is some confusion between TakeSelection() and AddSelection(). TakeSelection should first clear all other selections within the same widget. In addition, there is a naming conflict between nsIAccessible::AddSelection/RemoveSelection and the same method names in nsIAccessibleText. This makes it impossible to support both nsIAccessible and nsIAccessibleText in the same class.
Created attachment 225729 [details] [diff] [review] Addresses Mark's comments about tree items
Attachment #225729 - Flags: review?
Attachment #225727 - Flags: review?(pilgrim) → review-
Comment on attachment 225729 [details] [diff] [review] Addresses Mark's comments about tree items With additional fixes in nsXULTreeAccessible, this looks good.
Attachment #225729 - Flags: review? → review+
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.