Switch the Clang plugin to use RecursiveASTVisitor.

tools/clang/plugins/FindBadConstructsConsumer.cpp

Index: tools/clang/plugins/FindBadConstructsConsumer.cpp
diff --git a/tools/clang/plugins/FindBadConstructsConsumer.cpp b/tools/clang/plugins/FindBadConstructsConsumer.cpp
index f08ca712d5be731c8632b72dee76b3e0f969d666..4f281f1da0364df84b512ce544dd0c52c68dda1d 100644
--- a/tools/clang/plugins/FindBadConstructsConsumer.cpp
+++ b/tools/clang/plugins/FindBadConstructsConsumer.cpp
@@ -127,6 +127,13 @@ FindBadConstructsConsumer::FindBadConstructsConsumer(CompilerInstance& instance,
       DiagnosticsEngine::Note, kNoteProtectedNonVirtualDtor);
 }
 
+bool FindBadConstructsConsumer::VisitDecl(clang::Decl* decl) {
+  clang::TagDecl* tag_decl = dyn_cast<clang::TagDecl>(decl);
+  if (tag_decl && tag_decl->isCompleteDefinition())
+    CheckTag(tag_decl);
+  return true;
+}
+
 void FindBadConstructsConsumer::CheckChromeClass(SourceLocation record_location,
                                                  CXXRecordDecl* record) {
   bool implementation_file = InImplementationFile(record_location);