Bug 129814 – Regression: AX: Don't expose role or notifications for invalid menus

• Component: Accessibility
• Status: RESOLVED FIXED

[James Craig]

AX: Don't expose role or notifications for invalid menus

For example, on this page:
http://www.usmagazine.com/celebrity-news/news/andrew-garfield-refused-to-present-with-batkid-at-oscars-report-201463

Seems like this could be solved by having WebKit not fire the notification or example the menu role if the menu is invalid. It might be wise to not even expose it as an AXMenu if it's invalid. For example, this menu contains a link child (invalid) and no menu items (invalid).

<li class="" role="menu">
  <a class="dropdown-toggle" href="#" data-action="flag" data-role="flag" title="Flag as inappropriate">
    <i aria-hidden="true" class="icon icon-flag"></i>
  </a>
</li>

This appears to be fallout from bug 123133.

[James Craig]

<rdar://problem/16251311>

[James Craig]

s/or example the menu role/or expose the menu role/

[chris fleizach]

Created attachment 227154 [details]
patch

[Mario Sanchez Prada]

Comment on attachment 227154 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=227154&action=review

Lgtm. Please consider those suggestions below before landing

> Source/WebCore/accessibility/mac/WebAccessibilityObjectWrapperBase.mm:121
> +    m_object->updateChildrenIfNecessary();

For your consideration before landing: I wonder if you could fold this into updateBackingStore(), so you don't need to call it explicitly here, and other platforms get benefit from that too

> LayoutTests/ChangeLog:13
> +        * platform/mac/accessibility/table-with-aria-role-expected.txt:

You can move this test expectation now to LayoutTests/accessibility, as there should produce the same (platform independent) output now for all the platforms

[chris fleizach]

(In reply to comment #4)
> (From update of attachment 227154 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=227154&action=review
> 
> Lgtm. Please consider those suggestions below before landing
> 
> > Source/WebCore/accessibility/mac/WebAccessibilityObjectWrapperBase.mm:121
> > +    m_object->updateChildrenIfNecessary();
> 
> For your consideration before landing: I wonder if you could fold this into updateBackingStore(), so you don't need to call it explicitly here, and other platforms get benefit from that too
> 
> > LayoutTests/ChangeLog:13
> > +        * platform/mac/accessibility/table-with-aria-role-expected.txt:
> 
> You can move this test expectation now to LayoutTests/accessibility, as there should produce the same (platform independent) output now for all the platforms

Will do. Thanks!

[chris fleizach]

http://trac.webkit.org/changeset/166194