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

content/browser/accessibility/dump_accessibility_tree_helper_mac.mm

 // 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/dump_accessibility_tree_helper.h"
 
 #import <Cocoa/Cocoa.h>
 
 #include "base/file_path.h"
 #include "base/utf_string_conversions.h"
 #include "content/browser/accessibility/browser_accessibility_cocoa.h"
 #include "content/browser/accessibility/browser_accessibility_mac.h"
 
+namespace {
+string16 Format(BrowserAccessibility* node,
+                const char *prefix,
+                SEL selector,
+                const char *suffix) {
+  BrowserAccessibilityCocoa* cocoa_node = node->ToBrowserAccessibilityCocoa();
+  NSString* format_str =
+      [NSString stringWithFormat:@"%s%%@%s", prefix, suffix];
+  NSString* tmp = [NSString stringWithFormat:format_str,
+                   [cocoa_node performSelector:selector]];
+  return UTF8ToUTF16([tmp cStringUsingEncoding:NSUTF8StringEncoding]);
+}
+}
+
 void DumpAccessibilityTreeHelper::Initialize() {}
 
 string16 DumpAccessibilityTreeHelper::ToString(BrowserAccessibility* node,
                                                char* prefix) {
-  BrowserAccessibilityCocoa* cocoa_node = node->ToBrowserAccessibilityCocoa();
-  NSString* dump =
-      [NSString stringWithFormat:@"%s%@ "
-                                  "subrole=%@ "
-                                  "roleDescription='%@' "
-                                  "title='%@' "
-                                  "value='%@'\n",
-       prefix,
-       [cocoa_node role],
-       [cocoa_node subrole],
-       [cocoa_node roleDescription],
-       [cocoa_node title],
-       [cocoa_node value]];
-  std::string tempVal = [dump cStringUsingEncoding:NSUTF8StringEncoding];
-
-  return UTF8ToUTF16(tempVal);
+  StartLine();
+  Add(true, Format(node, "", @selector(role), ""));
+  Add(true, Format(node, "subrole=", @selector(subrole), ""));
+  Add(false, Format(node, "roleDescription='",
+                          @selector(roleDescription),
+                          "'"));
+  Add(true, Format(node, "title='", @selector(title), "'"));
+  Add(true, Format(node, "value='", @selector(value), "'"));
+  Add(false, Format(node, "description='", @selector(description), "'"));
+  Add(false, Format(node, "help='", @selector(help), "'"));
+  return ASCIIToUTF16(prefix) + FinishLine() + ASCIIToUTF16("\n");
 }
 
 const FilePath::StringType DumpAccessibilityTreeHelper::GetActualFileSuffix()
     const {
   return FILE_PATH_LITERAL("-actual-mac.txt");
 }
 
 const FilePath::StringType DumpAccessibilityTreeHelper::GetExpectedFileSuffix()
     const {
   return FILE_PATH_LITERAL("-expected-mac.txt");
 }
+
+const std::string DumpAccessibilityTreeHelper::GetAllowString() const {

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

nit: For simple/trivial accessors, do we want to name these as
"allow_filters()"? I realize it's not in the header, but perhaps more fitting
within the style guidelines.

+  return "@MAC-ALLOW:";
+}
+
+const std::string DumpAccessibilityTreeHelper::GetDenyString() const {
+  return "@MAC-DENY:";
+}