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..0696ffadf480ceb4f941a5a472d051412e9adf8f 100644
--- a/tools/clang/plugins/Options.h
+++ b/tools/clang/plugins/Options.h
@@ -8,21 +8,13 @@
 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;
+  bool check_implicit_copy_ctors = false;

[dcheng, 2016/02/17 06:09:14]

Let's add a comment about why this is needed and what the migration plan is for
removing this too.

[dcheng, 2016/02/17 06:09:49]

(For the migration plan, you can just link to the current bug)

[vmpstr, 2016/02/17 19:07:18]

Done.

 };
 
 }  // namespace chrome_checker