523789 - nsHTMLLiAccessible shouldn't be inherited from linkable accessible

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[alexander :surkov (:asurkov)]

Previously HTML li accessible was normal hyper text accessible, however it was turned into linkable accessible to handle registered click event listener in bug 301621. Since we started to expose 'click' action in nsAccessible class if proper event listener is registered then HTML li accessible don't need to be inherited from linkable accessible and we can remove this hack (because linkable accessible is designed to search action in parent chain including itself). As well this bug is part of reorganization work from bug 372131.

Comment 1

[alexander :surkov (:asurkov)]

Attachment: patch

Comment 2

[David Bolter [:davidb] (NeedInfo me for attention)]

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

r=me thanks; but would prefer nit addressed.

>+nsHTMLLIAccessible::
>+  nsHTMLLIAccessible(nsIDOMNode *aDOMNode, nsIWeakReference* aShell, 
>+                     const nsAString& aBulletText):
>+  nsHyperTextAccessibleWrap(aDOMNode, aShell)

Nit: Can you preserve nsHTMLLIAccessible::nsHTMLLIAccessible on the same line? I worry about static analysis tools and grep fails etc, when looking for constructors. Plus it looks weird to me. Is there precedent?

Comment 3

[alexander :surkov (:asurkov)]

(In reply to comment #2)
> (From update of attachment 407715 [details] [diff] [review])
> r=me thanks; but would prefer nit addressed.
> 
> >+nsHTMLLIAccessible::
> >+  nsHTMLLIAccessible(nsIDOMNode *aDOMNode, nsIWeakReference* aShell, 
> >+                     const nsAString& aBulletText):
> >+  nsHyperTextAccessibleWrap(aDOMNode, aShell)
> 
> Nit: Can you preserve nsHTMLLIAccessible::nsHTMLLIAccessible on the same line?
> I worry about static analysis tools and grep fails etc, when looking for
> constructors. Plus it looks weird to me. Is there precedent?

Almost everywhere in accessibility. Do they fail actually?

Comment 4

[David Bolter [:davidb] (NeedInfo me for attention)]

If it is done everywhere already, then that's fine. Do you like it?

(I doubt they fail if we already do this)

Comment 5

[alexander :surkov (:asurkov)]

(In reply to comment #4)
> If it is done everywhere already, then that's fine. Do you like it?

yep, it allows to keep big arguments list in more nice way

Comment 6

[alexander :surkov (:asurkov)]

landed on 1.9.3 - http://hg.mozilla.org/mozilla-central/rev/3c17a37fd4d2