Improvements to our windows accessibility tree testing.

content/browser/accessibility/accessibility_tree_formatter_win.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/accessibility/accessibility_tree_formatter.h"
 
 #include <oleacc.h>
 #include <stddef.h>
 #include <stdint.h>
 
@@ skipping 154 matching lines @@
     const BrowserAccessibility& node, base::DictionaryValue* dict) {
   dict->SetInteger("id", node.GetId());
   BrowserAccessibilityWin* ax_object =
       ToBrowserAccessibilityWin(const_cast<BrowserAccessibility*>(&node));
   DCHECK(ax_object);
 
   VARIANT variant_self;
   variant_self.vt = VT_I4;
   variant_self.lVal = CHILDID_SELF;
 
-  dict->SetString("role",
-                  IAccessible2RoleToString(ax_object->GetCOM()->ia2_role()));
+  long ia2_role = 0;
+  if (SUCCEEDED(ax_object->GetCOM()->role(&ia2_role))) {
+    dict->SetString("role", IAccessible2RoleToString(ia2_role));
+  }
 
   base::win::ScopedBstr temp_bstr;
   // If S_FALSE it means there is no name
   if (S_OK ==
       ax_object->GetCOM()->get_accName(variant_self, temp_bstr.Receive())) {
     base::string16 name = base::string16(temp_bstr, temp_bstr.Length());
 
     // Ignore a JAWS workaround where the name of a document is " ".
-    if (name != L" " || ax_object->GetCOM()->ia2_role() != ROLE_SYSTEM_DOCUMENT)
+    if (name != L" " || ia2_role != ROLE_SYSTEM_DOCUMENT)
       dict->SetString("name", name);
   }
   temp_bstr.Reset();
 
   if (SUCCEEDED(
           ax_object->GetCOM()->get_accValue(variant_self, temp_bstr.Receive())))
     dict->SetString("value", base::string16(temp_bstr, temp_bstr.Length()));
   temp_bstr.Reset();
 
   std::vector<base::string16> state_strings;
-  int32_t ia_state = ax_object->GetCOM()->ia_state();
+
+  int32_t ia_state = 0;
+  VARIANT ia_state_variant;
+  if (ax_object->GetCOM()->get_accState(variant_self, &ia_state_variant) ==
+          S_OK &&
+      ia_state_variant.vt == VT_I4) {
+    ia_state = ia_state_variant.intVal;
+  }
 
   // Avoid flakiness: these states depend on whether the window is focused
   // and the position of the mouse cursor.
   ia_state &= ~STATE_SYSTEM_HOTTRACKED;
   ia_state &= ~STATE_SYSTEM_OFFSCREEN;
 
-  IAccessibleStateToStringVector(ia_state, &state_strings);
-  IAccessible2StateToStringVector(ax_object->GetCOM()->ia2_state(),
-                                  &state_strings);
-  std::unique_ptr<base::ListValue> states(new base::ListValue());
-  for (const base::string16& state_string : state_strings)
-    states->AppendString(base::UTF16ToUTF8(state_string));
-  dict->Set("states", std::move(states));
+  // For testing, having the focused state may also cause flakiness if the
+  // window isn't in the foreground.
+  ia_state &= ~STATE_SYSTEM_FOCUSED;
 
-  const std::vector<base::string16>& ia2_attributes =
-      ax_object->GetCOM()->ia2_attributes();
-  std::unique_ptr<base::ListValue> attributes(new base::ListValue());
-  for (const base::string16& ia2_attribute : ia2_attributes)
-    attributes->AppendString(base::UTF16ToUTF8(ia2_attribute));
-  dict->Set("attributes", std::move(attributes));
+  AccessibleStates states;
+  if (ax_object->GetCOM()->get_states(&states) == S_OK) {
+    IAccessibleStateToStringVector(ia_state, &state_strings);
+    IAccessible2StateToStringVector(states, &state_strings);
+    std::unique_ptr<base::ListValue> states(new base::ListValue());
+    for (const base::string16& state_string : state_strings)
+      states->AppendString(base::UTF16ToUTF8(state_string));
+    dict->Set("states", std::move(states));
+  }
+
+  if (ax_object->GetCOM()->get_attributes(temp_bstr.Receive()) == S_OK) {
+    // get_attributes() returns a semicolon delimited string. Turn it into a
+    // ListValue
+    std::vector<base::string16> ia2_attributes = base::SplitString(
+        base::string16(temp_bstr, temp_bstr.Length()), base::string16(1, ';'),
+        base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL);
+
+    std::unique_ptr<base::ListValue> attributes(new base::ListValue());
+    for (const base::string16& ia2_attribute : ia2_attributes)
+      attributes->AppendString(base::UTF16ToUTF8(ia2_attribute));
+    dict->Set("attributes", std::move(attributes));
+  }
+  temp_bstr.Reset();
 
   ax_object->GetCOM()->ComputeStylesIfNeeded();
   const std::map<int, std::vector<base::string16>>& ia2_text_attributes =
       ax_object->GetCOM()->offset_to_text_attributes();
   std::unique_ptr<base::ListValue> text_attributes(new base::ListValue());
   for (const auto& style_span : ia2_text_attributes) {
     int start_offset = style_span.first;
     text_attributes->AppendString("offset:" + base::IntToString(start_offset));
     for (const base::string16& text_attribute : style_span.second)
       text_attributes->AppendString(base::UTF16ToUTF8(text_attribute));
@@ skipping 230 matching lines @@
 
 const std::string AccessibilityTreeFormatterWin::GetAllowString() {
   return "@WIN-ALLOW:";
 }
 
 const std::string AccessibilityTreeFormatterWin::GetDenyString() {
   return "@WIN-DENY:";
 }
 
 }  // namespace content