Allow filters in accessibility tests to specify which attributes to check.

content/test/accessibility_test_utils_win.cc

Index: content/test/accessibility_test_utils_win.cc
diff --git a/content/test/accessibility_test_utils_win.cc b/content/test/accessibility_test_utils_win.cc
index eaeacaae8c367fbd01edeb8eb3fb0eb174652f46..9c3f1879307196a781dd07665c118b3421766ad2 100644
--- a/content/test/accessibility_test_utils_win.cc
+++ b/content/test/accessibility_test_utils_win.cc
@@ -230,38 +230,36 @@ string16 IAccessible2RoleToString(int32 ia_role) {
   return AccessibilityRoleStateMap::GetInstance()->ia2_role_string_map[ia_role];
 }
 
-string16 IAccessibleStateToString(int32 ia_state) {
-  string16 state_str;
+void IAccessibleStateToStringVector(int32 ia_state,
+                                    std::vector<string16>* result) {
   const std::map<int32, string16>& state_string_map =
       AccessibilityRoleStateMap::GetInstance()->ia_state_string_map;
   std::map<int32, string16>::const_iterator it;
-
-  for (it = state_string_map.begin();
-       it != state_string_map.end();
-       ++it) {
-    if (it->first & ia_state) {
-      if (!state_str.empty())
-        state_str += L",";
-      state_str += it->second;
-    }
+  for (it = state_string_map.begin(); it != state_string_map.end(); ++it) {
+    if (it->first & ia_state)
+      result->push_back(it->second);
   }
-  return state_str;
 }
 
-string16 IAccessible2StateToString(int32 ia2_state) {
-  string16 state_str;
+string16 IAccessibleStateToString(int32 ia_state) {
+  std::vector<string16> strings;
+  IAccessibleStateToStringVector(ia_state, &strings);
+  return JoinString(strings, ',');
+}
+
+void IAccessible2StateToStringVector(int32 ia2_state,
+                                     std::vector<string16>* result) {
   const std::map<int32, string16>& state_string_map =
       AccessibilityRoleStateMap::GetInstance()->ia2_state_string_map;
   std::map<int32, string16>::const_iterator it;
-
-  for (it = state_string_map.begin();
-       it != state_string_map.end();
-       ++it) {
-    if (it->first & ia2_state) {
-      if (!state_str.empty())
-        state_str += L",";
-      state_str += it->second;
-    }
+  for (it = state_string_map.begin(); it != state_string_map.end(); ++it) {
+    if (it->first & ia2_state)
+      result->push_back(it->second);

[David Tseng, 2012/07/31 17:04:17]

nit: This seems repetitive with above (seems like the map could be passed as a
param).

   }
-  return state_str;
+}
+
+string16 IAccessible2StateToString(int32 ia2_state) {
+  std::vector<string16> strings;
+  IAccessible2StateToStringVector(ia2_state, &strings);
+  return JoinString(strings, ',');
 }