Summary tag is exposed with wrong IA2 Role

Source/core/accessibility/AXRenderObject.cpp

Index: Source/core/accessibility/AXRenderObject.cpp
diff --git a/Source/core/accessibility/AXRenderObject.cpp b/Source/core/accessibility/AXRenderObject.cpp
index 8ee9d6547dc7c27e12223599572869565d9e5858..15b0a3bc16890c0844e79e89e686dbcc9597d03a 100644
--- a/Source/core/accessibility/AXRenderObject.cpp
+++ b/Source/core/accessibility/AXRenderObject.cpp
@@ -280,6 +280,11 @@ AccessibilityRole AXRenderObject::determineAccessibilityRole()
         return buttonRoleType();
     if (isHTMLDetailsElement(node))
         return DetailsRole;
+    if (isHTMLSummaryElement(node)) {
+        if (isHTMLDetailsElement(m_renderer->parent()->node()))

[Mike West, 2014/09/25 13:08:01]

I think you need to null-check parent() here. Otherwise detached nodes will
crash.

[dmazzoni, 2014/09/25 15:19:49]

Also, I think it'd be more correct to check node->parentElement rather than
m_renderer->parent()->node.

The render tree can differ from the node tree in subtle ways. If the details
element has multiple children and some are inline and some are block, the render
tree may have an anonymous block in-between, so the details would not
necessarily be the direct parent of the summary element.

+            return DisclosureTriangleRole;
+        return UnknownRole;
+    }
     if (isHTMLLegendElement(node))
         return LegendRole;
     if (m_renderer->isText())