clang-plugin: Enable RecursiveASTVisitor approach by default.

tools/clang/plugins/Options.h

Index: tools/clang/plugins/Options.h
diff --git a/tools/clang/plugins/Options.h b/tools/clang/plugins/Options.h
index bb5857f0d74ec688dc158367b24b0a1c7643d4c2..2ec267aaafbe1075092707fc03cff77b9164cf3f 100644
--- a/tools/clang/plugins/Options.h
+++ b/tools/clang/plugins/Options.h
@@ -8,21 +8,15 @@
 namespace chrome_checker {
 
 struct Options {
-  Options()
-      : check_base_classes(false),
-        enforce_in_pdf(false),
-        enforce_in_thirdparty_webkit(false),
-        check_enum_last_value(false),
-        with_ast_visitor(false),
-        check_templates(false) {}
-
-  bool check_base_classes;
-  bool enforce_in_pdf;
-  bool enforce_in_thirdparty_webkit;  // Use in Blink code itself
-  bool check_enum_last_value;
-  bool with_ast_visitor;
-  bool check_templates;
+  bool check_base_classes = false;
+  bool enforce_in_pdf = false;
+  bool enforce_in_thirdparty_webkit = false;  // Use in Blink code itself
+  bool check_enum_last_value = false;
+  bool check_templates = false;
   bool follow_macro_expansion = false;
+  // This is needed during the migration from ASTConsumer approach to the
+  // RecursiveASTVisitor approach. See crbug.com/436357 for details.

[dcheng, 2016/02/17 20:15:20]

Nit: https://crbug.com, it makes auto-linkification on codesearch work too.

[vmpstr, 2016/02/17 21:22:53]

Done.

+  bool check_implicit_copy_ctors = false;
 };
 
 }  // namespace chrome_checker