Chromium Code Reviews| Index: Source/core/accessibility/AXNodeObject.cpp |
| diff --git a/Source/core/accessibility/AXNodeObject.cpp b/Source/core/accessibility/AXNodeObject.cpp |
| index 28f096c7cddb160b66987da39743d3398478b60f..6008b2848328e01e21097affbf8e2d3ee25377d4 100644 |
| --- a/Source/core/accessibility/AXNodeObject.cpp |
| +++ b/Source/core/accessibility/AXNodeObject.cpp |
| @@ -204,12 +204,12 @@ AccessibilityRole AXNodeObject::determineAccessibilityRole() |
| HTMLInputElement& input = toHTMLInputElement(*node()); |
| const AtomicString& type = input.type(); |
| if (type == InputTypeNames::button) { |
| - if (node()->parentNode() && isHTMLMenuElement(node()->parentNode())) |
| + if (node()->parentNode() && (isHTMLMenuElement(node()->parentNode()) || parentObject()->roleValue() == MenuRole)) |
|
dmazzoni
2014/10/06 15:25:43
It's possible parentObject() could return null, so
|
| return MenuItemRole; |
| return buttonRoleType(); |
| } |
| if (type == InputTypeNames::checkbox) { |
| - if (node()->parentNode() && isHTMLMenuElement(node()->parentNode())) |
| + if (node()->parentNode() && (isHTMLMenuElement(node()->parentNode()) || parentObject()->roleValue() == MenuRole)) |
| return CheckBoxMenuItemRole; |
| return CheckBoxRole; |
| } |