rewrite_to_chrome_style: Blacklist InspectorAgent::disable

tools/clang/rewrite_to_chrome_style/RewriteToChromeStyle.cpp

 // Copyright 2015 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.
 //
 // Changes Blink-style names to Chrome-style names. Currently transforms:
 //   fields:
 //     int m_operationCount => int operation_count_
 //   variables (including parameters):
 //     int mySuperVariable => int my_super_variable
 //   constants:
@@ skipping 126 matching lines @@
   return inner_matcher.matches(decl, finder, builder);
 }
 
 AST_MATCHER_P(clang::CXXMethodDecl,
               includeAllOverriddenMethods,
               clang::ast_matchers::internal::Matcher<clang::CXXMethodDecl>,
               InnerMatcher) {
   return MatchAllOverriddenMethods(Node, InnerMatcher, Finder, Builder);
 }
 
+bool IsMethodOverrideOf(const clang::CXXMethodDecl& decl,
+                        const char* class_name) {
+  if (decl.getParent()->getQualifiedNameAsString() == class_name)
+    return true;
+  for (auto it = decl.begin_overridden_methods();
+       it != decl.end_overridden_methods(); ++it) {
+    if (IsMethodOverrideOf(**it, class_name))
+      return true;
+  }
+  return false;
+}
+
 bool IsBlacklistedMethod(const clang::CXXMethodDecl& decl) {
   if (decl.isStatic())
     return false;
 
   clang::StringRef name = decl.getName();
 
   // These methods should never be renamed.
   static const char* kBlacklistMethods[] = {"trace", "lock", "unlock",
                                             "try_lock"};
   for (const auto& b : kBlacklistMethods) {
     if (name == b)
       return true;
   }
 
   // Iterator methods shouldn't be renamed to work with stl and range-for
   // loops.
   std::string ret_type = decl.getReturnType().getAsString();
-  if (ret_type.find("iterator") == std::string::npos &&
-      ret_type.find("Iterator") == std::string::npos)
-    return false;
-  static const char* kIteratorBlacklist[] = {"begin", "end", "rbegin", "rend"};
-  for (const auto& b : kIteratorBlacklist) {
-    if (name == b)
-      return true;
+  if (ret_type.find("iterator") != std::string::npos ||
+      ret_type.find("Iterator") != std::string::npos) {
+    static const char* kIteratorBlacklist[] = {"begin", "end", "rbegin",
+                                               "rend"};
+    for (const auto& b : kIteratorBlacklist) {
+      if (name == b)
+        return true;
+    }
   }
 
+  // Subclasses of InspectorAgent will subclass "disable()" from both blink and
+  // from gen/, which is problematic, but DevTools folks don't want to rename
+  // it or split this up. So don't rename it at all.
+  if (name.equals("disable") &&
+      IsMethodOverrideOf(decl, "blink::InspectorAgent"))
+    return true;
+
   return false;
 }
 
 AST_MATCHER(clang::CXXMethodDecl, isBlacklistedMethod) {
   return IsBlacklistedMethod(Node);
 }
 
 // Helper to convert from a camelCaseName to camel_case_name. It uses some
 // heuristics to try to handle acronyms in camel case names correctly.
 std::string CamelCaseToUnderscoreCase(StringRef input) {
@@ skipping 593 matching lines @@
   for (const auto& r : replacements) {
     std::string replacement_text = r.getReplacementText().str();
     std::replace(replacement_text.begin(), replacement_text.end(), '\n', '\0');
     llvm::outs() << "r:::" << r.getFilePath() << ":::" << r.getOffset()
                  << ":::" << r.getLength() << ":::" << replacement_text << "\n";
   }
   llvm::outs() << "==== END EDITS ====\n";
 
   return 0;
 }