Implement aria properties setsize and posinset.

Source/modules/accessibility/AXObject.cpp

Index: Source/modules/accessibility/AXObject.cpp
diff --git a/Source/modules/accessibility/AXObject.cpp b/Source/modules/accessibility/AXObject.cpp
index 55c4a517c4e225c8aea22265ddab2e4bd875aa22..9845bdede0ac6edd8ed90859619e11cd4169be99 100644
--- a/Source/modules/accessibility/AXObject.cpp
+++ b/Source/modules/accessibility/AXObject.cpp
@@ -707,7 +707,16 @@ bool AXObject::supportsARIAAttributes() const
         || 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
@@ -731,6 +740,16 @@ void AXObject::ariaTreeRows(AccessibilityChildrenVector& 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();