Add 4 ARIA 1.1 attributes using new sparse attribute interface.

third_party/WebKit/Source/modules/accessibility/InspectorAccessibilityAgent.cpp

Index: third_party/WebKit/Source/modules/accessibility/InspectorAccessibilityAgent.cpp
diff --git a/third_party/WebKit/Source/modules/accessibility/InspectorAccessibilityAgent.cpp b/third_party/WebKit/Source/modules/accessibility/InspectorAccessibilityAgent.cpp
index 235cd9b5e1263f50375f057e118ea7b78008e28d..c08101a17f4d1e58cff486183af350bd272c1d6a 100644
--- a/third_party/WebKit/Source/modules/accessibility/InspectorAccessibilityAgent.cpp
+++ b/third_party/WebKit/Source/modules/accessibility/InspectorAccessibilityAgent.cpp
@@ -341,9 +341,24 @@ class SparseAttributeAXPropertyAdapter
   Member<AXObject> m_axObject;
   protocol::Array<AXProperty>& m_properties;
 
-  void addBoolAttribute(AXBoolAttribute attribute, bool value) {}
+  void addBoolAttribute(AXBoolAttribute attribute, bool value) {
+    // TODO

[aboxhall, 2017/01/27 01:41:22]

TODO

[dmazzoni, 2017/01/31 00:31:54]

Nothing to add for this particular change. I changed the comment to say that
we should implement this when we add our first sparse boolean attribute.

+  }
 
-  void addStringAttribute(AXStringAttribute attribute, const String& value) {}
+  void addStringAttribute(AXStringAttribute attribute, const String& value) {
+    switch (attribute) {
+      case AXStringAttribute::AriaKeyShortcuts:
+        m_properties.addItem(
+            createProperty(AXGlobalStatesEnum::Keyshortcuts,
+                           createValue(value, AXValueTypeEnum::String)));
+        break;
+      case AXStringAttribute::AriaRoleDescription:
+        m_properties.addItem(
+            createProperty(AXGlobalStatesEnum::Roledescription,
+                           createValue(value, AXValueTypeEnum::String)));
+        break;
+    }
+  }
 
   void addObjectAttribute(AXObjectAttribute attribute, AXObject& object) {
     switch (attribute) {
@@ -352,6 +367,11 @@ class SparseAttributeAXPropertyAdapter
             createProperty(AXRelationshipAttributesEnum::Activedescendant,
                            createRelatedNodeListValue(object)));
         break;
+      case AXObjectAttribute::AriaErrorMessage:
+        m_properties.addItem(
+            createProperty(AXRelationshipAttributesEnum::Errormessage,
+                           createRelatedNodeListValue(object)));
+        break;
     }
   }
 
@@ -363,6 +383,11 @@ class SparseAttributeAXPropertyAdapter
             AXRelationshipAttributesEnum::Controls, objects, aria_controlsAttr,
             *m_axObject));
         break;
+      case AXObjectVectorAttribute::AriaDetails:
+        m_properties.addItem(createRelatedNodeListProperty(
+            AXRelationshipAttributesEnum::Details, objects, aria_controlsAttr,
+            *m_axObject));
+        break;
       case AXObjectVectorAttribute::AriaFlowTo:
         m_properties.addItem(createRelatedNodeListProperty(
             AXRelationshipAttributesEnum::Flowto, objects, aria_flowtoAttr,