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

tools/clang/rewrite_to_chrome_style/RewriteToChromeStyle.cpp

Index: tools/clang/rewrite_to_chrome_style/RewriteToChromeStyle.cpp
diff --git a/tools/clang/rewrite_to_chrome_style/RewriteToChromeStyle.cpp b/tools/clang/rewrite_to_chrome_style/RewriteToChromeStyle.cpp
index 8b0af6a8cc8af140f162c1ee09f44be901ea29ef..310da83853f393841cd8fd0c472826588e206a88 100644
--- a/tools/clang/rewrite_to_chrome_style/RewriteToChromeStyle.cpp
+++ b/tools/clang/rewrite_to_chrome_style/RewriteToChromeStyle.cpp
@@ -428,7 +428,10 @@ bool IsBlacklistedInstanceMethodName(llvm::StringRef name) {
 
       // 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",

[dcheng, 2017/05/08 18:56:12]

We can revert this file altogether for now as well, right?

[Łukasz Anforowicz, 2017/05/08 19:06:03]

Done.  In theory we could have kept addition of "clear", "find", "swap" above,
but 1) this is not necessary, 2) probably belongs in another CL and 3) is tricky
(blanket blacklisting is not quite right - e.g. probably okay for
Member<T>::Clear to be uppercased).

   };
   for (const auto& b : kBlacklistedNames) {
     if (name == b)