Chromium side Implementation to expose aria properties - setsize & posinset

content/browser/accessibility/browser_accessibility_cocoa.mm

Index: content/browser/accessibility/browser_accessibility_cocoa.mm
diff --git a/content/browser/accessibility/browser_accessibility_cocoa.mm b/content/browser/accessibility/browser_accessibility_cocoa.mm
index 7db4fc7fa29617bcb18363849b9908c754bcd916..5b61e6fcbbf4cfbb126994ae948d2526d8d639e5 100644
--- a/content/browser/accessibility/browser_accessibility_cocoa.mm
+++ b/content/browser/accessibility/browser_accessibility_cocoa.mm
@@ -108,6 +108,8 @@ NSDictionary* attributeToMethodNameMap = nil;
     { @"AXARIAAtomic", @"ariaAtomic" },
     { @"AXARIABusy", @"ariaBusy" },
     { @"AXARIALive", @"ariaLive" },
+    { @"AXARIAPosInSet", @"ariaPosInSet" },
+    { @"AXARIASetSize", @"ariaSetSize" },
     { @"AXARIARelevant", @"ariaRelevant" },
     { @"AXDropEffects", @"dropeffect" },
     { @"AXGrabbed", @"grabbed" },
@@ -170,6 +172,16 @@ NSDictionary* attributeToMethodNameMap = nil;
       browserAccessibility_, ui::AX_ATTR_LIVE_RELEVANT);
 }
 
+- (NSNumber*)ariaPosInSet {
+  return [NSNumber numberWithInt:
+      browserAccessibility_->GetIntAttribute(ui::AX_ATTR_POS_IN_SET)];
+}
+
+- (NSNumber*)ariaSetSize {
+  return [NSNumber numberWithInt:
+      browserAccessibility_->GetIntAttribute(ui::AX_ATTR_SET_SIZE)];
+}
+
 // Returns an array of BrowserAccessibilityCocoa objects, representing the
 // accessibility children of this object.
 - (NSArray*)children {
@@ -1311,6 +1323,8 @@ NSDictionary* attributeToMethodNameMap = nil;
       NSAccessibilityValueAttribute,
       NSAccessibilityWindowAttribute,
       @"AXAccessKey",
+      @"AXARIAPosInSet",

[dmazzoni, 2015/04/21 05:56:21]

I think we should only include these if they're relevant. It doesn't make sense
to expose posinset/setsize on all nodes.

I'd do it by checking
browserAccessibility_->HasIntAttribute(ui::AX_ATTR_SET_SIZE) and only add
support for these attributes if that attribute is present, kind of like the
other ones added below.

+      @"AXARIASetSize",
       @"AXInvalid",
       @"AXVisited",
       nil];