Blink rename follow-up: Use Chromium style in network_instrumentation namespace.

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 410 matching lines @@
 bool IsBlacklistedInstanceMethodName(llvm::StringRef name) {
   static const char* kBlacklistedNames[] = {
       // We should avoid renaming the method names listed below, because
       // 1. They are used in templated code (e.g. in <algorithms>)
       // 2. They (begin+end) are used in range-based for syntax sugar
       //    - for (auto x : foo) { ... }  // <- foo.begin() will be called.
       "begin", "end", "rbegin", "rend", "lock", "unlock", "try_lock",
 
       // https://crbug.com/672902: Should not rewrite names that mimick methods
       // from std library.
-      "at", "back", "empty", "erase", "front", "insert", "length", "size",
+      "at", "back",
+      "clear"
+      "empty",
+      "erase", "find", "front", "insert", "length", "size", "swap",

[Łukasz Anforowicz, 2017/05/08 17:09:45]

Above formatting was produced by clang-format.  I've opened
https://bugs.llvm.org//show_bug.cgi?id=32969.

   };
   for (const auto& b : kBlacklistedNames) {
     if (name == b)
       return true;
   }
   return false;
 }
 
 bool IsBlacklistedMethodName(llvm::StringRef name) {
   return IsBlacklistedFunctionName(name) ||
@@ skipping 1083 matching lines @@
   CommonOptionsParser options(argc, argv, category);
   MethodBlocklist method_blocklist(blocklisted_methods_file);
   clang::tooling::ClangTool tool(options.getCompilations(),
                                  options.getSourcePathList());
 
   MatchFinder match_finder;
   std::set<Replacement> replacements;
 
   // Blink namespace matchers ========
   auto blink_namespace_decl =
-      namespaceDecl(anyOf(hasName("blink"), hasName("WTF")),
+      namespaceDecl(anyOf(hasName("blink"), hasName("WTF"),
+                          hasName("network_instrumentation")),
                     hasParent(translationUnitDecl()));
   auto protocol_namespace_decl =
       namespaceDecl(hasName("protocol"),
                     hasParent(namespaceDecl(hasName("blink"),
                                             hasParent(translationUnitDecl()))));
 
   // Given top-level compilation unit:
   //   namespace WTF {
   //     void foo() {}
   //   }
@@ skipping 406 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;
 }