clang plugin: Put inner classes bugfix behind a flag.

tools/clang/plugins/ChromeClassTester.cpp

 // 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.
 
 // A general interface for filtering and only acting on classes in Chromium C++
 // code.
 
 #include "ChromeClassTester.h"
 
 #include <sys/param.h>
@@ skipping 18 matching lines @@
 
 bool ends_with(const std::string& one, const std::string& two) {
   if (two.size() > one.size())
     return false;
 
   return one.compare(one.size() - two.size(), two.size(), two) == 0;
 }
 
 }  // namespace
 
-ChromeClassTester::ChromeClassTester(CompilerInstance& instance)
+ChromeClassTester::ChromeClassTester(CompilerInstance& instance,
+                                     bool check_inner_classes)
     : instance_(instance),
-      diagnostic_(instance.getDiagnostics()) {
+      diagnostic_(instance.getDiagnostics()),
+      check_inner_classes_(check_inner_classes) {
   BuildBannedLists();
 }
 
 ChromeClassTester::~ChromeClassTester() {}
 
 void ChromeClassTester::HandleTagDeclDefinition(TagDecl* tag) {
-  // Defer processing of this tag until its containing top-level
-  // declaration has been fully parsed. See crbug.com/136863.
-  pending_class_decls_.push_back(tag);
+  if (check_inner_classes_) {
+    // Defer processing of this tag until its containing top-level
+    // declaration has been fully parsed. See crbug.com/136863.
+    pending_class_decls_.push_back(tag);
+  } else {
+    CheckTag(tag);
+  }
 }
 
 bool ChromeClassTester::HandleTopLevelDecl(DeclGroupRef group_ref) {
   for (size_t i = 0; i < pending_class_decls_.size(); ++i)
     CheckTag(pending_class_decls_[i]);
   pending_class_decls_.clear();
 
   return true;  // true means continue parsing.
 }
 
@@ skipping 226 matching lines @@
   PresumedLoc ploc = source_manager.getPresumedLoc(spelling_location);
   if (ploc.isInvalid()) {
     // If we're in an invalid location, we're looking at things that aren't
     // actually stated in the source.
     return false;
   }
 
   *filename = ploc.getFilename();
   return true;
 }