Update methods requiring Get prefix to avoid collisions with Blink rewrite.

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 196 matching lines @@
   return false;
 }
 
 bool IsBlacklistedFunctionName(llvm::StringRef name) {
   // https://crbug.com/672902: Method names with an underscore are typically
   // mimicked after std library / are typically not originating from Blink.
   // Do not rewrite such names (like push_back, emplace_back, etc.).
   if (name.find('_') != llvm::StringRef::npos)
     return true;
 
-  // https://crbug.com/677166: Have to avoid renaming |hash| -> |Hash| to avoid
-  // colliding with a struct already named |Hash|.
-  return name == "hash";
+  return false;
 }
 
 bool IsBlacklistedFreeFunctionName(llvm::StringRef name) {
   // swap() functions should match the signature of std::swap for ADL tricks.
   return name == "swap";
 }
 
 bool IsBlacklistedInstanceMethodName(llvm::StringRef name) {
   static const char* kBlacklistedNames[] = {
       // We should avoid renaming the method names listed below, because
@@ skipping 121 matching lines @@
   // If the expression can be evaluated at compile time, then it should have a
   // kFoo style name. Otherwise, not.
   return initializer->isEvaluatable(context);
 }
 
 AST_MATCHER_P(clang::QualType, hasString, std::string, ExpectedString) {
   return ExpectedString == Node.getAsString();
 }
 
 bool ShouldPrefixFunctionName(const std::string& old_method_name) {
   // Methods that are named similarily to a type - they should be prefixed

[danakj, 2017/01/04 18:44:51]

Can you fix this comment - "Method" is incorrect here as this is more widely
applicable as per Lucasz' comment. Then LGTM

[nasko, 2017/01/04 19:03:02]

Done.

   // with a "get" prefix.
   static const char* kConflictingMethods[] = {
       "animationWorklet",
       "audioWorklet",
       "binaryType",
       "blob",
       "channelCountMode",
       "color",
       "counterDirectives",
       "document",
       "emptyChromeClient",
       "emptyEditorClient",
       "emptySpellCheckerClient",
       "entryType",
       "error",
       "fileUtilities",
       "font",
       "frame",
       "frameBlameContext",
+      "frontend",
+      "hash",
+      "heapObjectHeader",
       "iconURL",
       "inputMethodController",
       "inputType",
       "layout",
       "layoutBlock",
       "layoutObject",
       "layoutSize",
       "length",
       "lineCap",
+      "lineEndings",
       "lineJoin",
+      "listItems",
       "matchedProperties",
+      "midpointState",
+      "mouseEvent",
       "name",
       "navigationType",
       "node",
       "outcome",
       "pagePopup",
       "paintWorklet",
       "path",
       "processingInstruction",
       "readyState",
+      "relList",
+      "resource",
       "response",
       "sandboxSupport",
       "screenInfo",
       "scrollAnimator",
       "settings",
       "signalingState",
       "state",
       "string",
+      "styleSheet",
       "text",
       "textAlign",
       "textBaseline",
       "theme",
+      "thread",
       "timing",
       "topLevelBlameContext",
+      "vector",
       "widget",
+      "wordBoundaries",
+      "wrapperTypeInfo",
   };
   for (const auto& conflicting_method : kConflictingMethods) {
     if (old_method_name == conflicting_method)
       return true;
   }
 
   return false;
 }
 
 AST_MATCHER(clang::FunctionDecl, shouldPrefixFunctionName) {
@@ skipping 966 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;
 }