Implement aria properties setsize and posinset.

Source/modules/accessibility/AXObject.cpp

 /*
  * Copyright (C) 2008, 2009, 2011 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 689 matching lines @@
     return !getAttribute(aria_pressedAttr).isEmpty();
 }
 
 bool AXObject::supportsARIAAttributes() const
 {
     return isLiveRegion()
         || supportsARIADragging()
         || supportsARIADropping()
         || supportsARIAFlowTo()
         || supportsARIAOwns()
-        || hasAttribute(aria_labelAttr);
+        || hasAttribute(aria_atomicAttr)

[dmazzoni, 2015/04/09 06:21:41]

Calling hasAttribute so many times in a row couldn't possibly be efficient.
There are now more than a dozen attributes we're checking but *most* elements
will have very few attributes, and no ARIA attributes.

I think it'd be better to iterate over all of the attributes and use a switch()
to see if any of them is an ARIA attribute or the "role" attribute, then we
should cache this computation.

Could you file a bug to do that in another change?

+        || hasAttribute(aria_busyAttr)
+        || hasAttribute(aria_controlsAttr)
+        || hasAttribute(aria_describedbyAttr)
+        || hasAttribute(aria_disabledAttr)
+        || hasAttribute(aria_haspopupAttr)
+        || hasAttribute(aria_invalidAttr)
+        || hasAttribute(aria_labelAttr)
+        || hasAttribute(aria_labelledbyAttr)
+        || hasAttribute(aria_relevantAttr);
 }
 
 bool AXObject::supportsRangeValue() const
 {
     return isProgressIndicator()
         || isMeter()
         || isSlider()
         || isScrollbar()
         || isSpinButton();
 }
 
 void AXObject::ariaTreeRows(AccessibilityChildrenVector& result)
 {
     for (const auto& child : children()) {
         // Add tree items as the rows.
         if (child->roleValue() == TreeItemRole)
             result.append(child);
 
         // Now see if this item also has rows hiding inside of it.
         child->ariaTreeRows(result);
     }
 }
 
+int AXObject::ariaPosInSet() const
+{
+    return getAttribute(aria_posinsetAttr).toInt();
+}
+
+int AXObject::ariaSetSize() const
+{
+    return getAttribute(aria_setsizeAttr).toInt();
+}
+
 bool AXObject::isLiveRegion() const
 {
     const AtomicString& liveRegion = liveRegionStatus();
     return equalIgnoringCase(liveRegion, "polite") || equalIgnoringCase(liveRegion, "assertive");
 }
 
 const AXObject* AXObject::liveRegionRoot() const
 {
     updateCachedAttributeValuesIfNeeded();
     return m_cachedLiveRegionRoot;
@@ skipping 556 matching lines @@
 }
 
 const AtomicString& AXObject::internalRoleName(AccessibilityRole role)
 {
     static const Vector<AtomicString>* internalRoleNameVector = createInternalRoleNameVector();
 
     return internalRoleNameVector->at(role);
 }
 
 } // namespace blink