Check whether aria-expanded attribute is defined

Source/core/accessibility/AXMenuList.cpp

Index: Source/core/accessibility/AXMenuList.cpp
diff --git a/Source/core/accessibility/AXMenuList.cpp b/Source/core/accessibility/AXMenuList.cpp
index 4d5588e1b00c03c7413906544482c519ca52510a..2b0283cad1726b42ec47d93eea45827bdb7d9872 100644
--- a/Source/core/accessibility/AXMenuList.cpp
+++ b/Source/core/accessibility/AXMenuList.cpp
@@ -87,6 +87,19 @@ bool AXMenuList::isCollapsed() const
     return !toRenderMenuList(m_renderer)->popupIsVisible();
 }
 
+AccessibilityExpanded AXMenuList::ariaExpanded() const
+{
+    AccessibilityExpanded expanded = getExpandedAttr();

[dmazzoni, 2014/10/29 07:46:59]

The ARIA spec says that ARIA shouldn't be allowed to override states that have
native semantics. So in this case, an AXMenuList is a native select element and
there's already a semantic meaning to whether it's expanded and collapsed, so
ARIA is not allowed to override it, that would only create confusion.

http://www.w3.org/TR/wai-aria-implementation/#mapping_conflicts

So basically I think this function should ignore the aria-expanded attribute and
just return whether the AXMenuList is really collapsed or expanded.

[je_julie(Not used), 2014/10/29 13:07:27]

Thanks for your comment. I updated it.
On 2014/10/29 07:46:59, dmazzoni wrote:

+
+    if (expanded)
+        return expanded;
+
+    if (isCollapsed())
+        return ExpandedCollapsed;
+
+    return ExpandedExpanded;
+}
+
 bool AXMenuList::canSetFocusAttribute() const
 {
     if (!node())