AX presentation role should be inherited to its required owned elements.

Source/modules/accessibility/AXLayoutObject.cpp

 /*
 * Copyright (C) 2008 Apple Inc. All rights reserved.
 *
 * Redistribution and use in source and binary forms, with or without
 * modification, are permitted provided that the following conditions
 * are met:
 *
 * 1.  Redistributions of source code must retain the above copyright
 *     notice, this list of conditions and the following disclaimer.
 * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 545 matching lines @@
     if (decision == IgnoreObject)
         return true;
 
     // If this element is within a parent that cannot have children, it should not be exposed.
     if (isDescendantOfBarrenParent())
         return true;
 
     if (roleValue() == IgnoredRole)
         return true;
 
-    if ((roleValue() == NoneRole || roleValue() == PresentationalRole) || inheritsPresentationalRole())
+    if (isPresentationRole(static_cast<const AXObject*>(this)))

[dmazzoni, 2015/03/31 16:35:23]

Can you do this without the cast?

If you do need a cast it should probably be const_cast instead of static_cast

[je_julie(Not used), 2015/04/01 12:51:28]

Because |this| is AXLayoutObject and parameter type is AXObject, I needed it.
But I removed parameter on new patchset.

         return true;
 
     // An ARIA tree can only have tree items and static text as children.
     if (!isAllowedChildOfTree())
         return true;
 
     // TODO: we should refactor this - but right now this is necessary to make
     // sure scroll areas stay in the tree.
     if (isAttachment())
         return false;
@@ skipping 1708 matching lines @@
 }
 
 bool AXLayoutObject::elementAttributeValue(const QualifiedName& attributeName) const
 {
     if (!m_layoutObject)
         return false;
 
     return equalIgnoringCase(getAttribute(attributeName), "true");
 }
 
-bool AXLayoutObject::inheritsPresentationalRole() const
-{
-    // ARIA states if an item can get focus, it should not be presentational.
-    if (canSetFocusAttribute())
-        return false;
-
-    // ARIA spec says that when a parent object is presentational, and it has required child elements,
-    // those child elements are also presentational. For example, <li> becomes presentational from <ul>.
-    // http://www.w3.org/WAI/PF/aria/complete#presentation
-    if (roleValue() != ListItemRole && roleValue() != ListMarkerRole)
-        return false;
-
-    AXObject* parent = parentObject();
-    if (!parent->isAXLayoutObject())
-        return false;
-
-    Node* elementNode = toAXLayoutObject(parent)->node();
-    if (!elementNode || !elementNode->isElementNode())
-        return false;
-
-    QualifiedName tagName = toElement(elementNode)->tagQName();
-    if (tagName != ulTag && tagName != olTag && tagName != dlTag)
-        return false;
-
-    if (parent->roleValue() == NoneRole || parent->roleValue() == PresentationalRole)
-        return ariaRoleAttribute() == UnknownRole;
-
-    return false;
-}
-
 LayoutRect AXLayoutObject::computeElementRect() const
 {
     LayoutObject* obj = m_layoutObject;
 
     if (!obj)
         return LayoutRect();
 
     if (obj->node()) // If we are a continuation, we want to make sure to use the primary layoutObject.
         obj = obj->node()->layoutObject();
 
@@ skipping 31 matching lines @@
         if (label && label->layoutObject()) {
             LayoutRect labelRect = axObjectCache()->getOrCreate(label)->elementRect();
             result.unite(labelRect);
         }
     }
 
     return result;
 }
 
 } // namespace blink