Prepending "Get" when type with conflicting name is hidden inside return type.

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 272 matching lines @@
   // instantiation of the template and see if they are all compile-time
   // isEvaluable().
   if (initializer->isInstantiationDependent())
     return false;
 
   // 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 GetNameForDecl(const clang::FunctionDecl& decl,
                     clang::ASTContext& context,
                     std::string& name) {
   name = decl.getName().str();
   name[0] = clang::toUppercase(name[0]);
 
-  // https://crbug.com/582312: Prepend "Get" if method name conflicts with type.
-  const clang::IdentifierInfo* return_type =
-      decl.getReturnType().getBaseTypeIdentifier();
-  if (return_type && return_type->getName() == name)
+  // Given
+  //   class Foo {};
+  //   using Bar = Foo;
+  //   Bar f1();  // <- |Bar| would be matched by hasString("Bar") below.
+  //   Bar f2();  // <- |Bar| would be matched by hasName("Foo") below.
+  // |type_with_same_name_as_function| matcher matches Bar and Foo return types.
+  auto type_with_same_name_as_function = qualType(anyOf(
+      hasString(name),  // hasString matches the type as spelled (Bar above).
+      hasDeclaration(namedDecl(hasName(name)))));  // hasDeclaration matches
+                                                   // resolved type (Foo above).
+  // |type_containing_same_name_as_function| matcher will match all of the
+  // return types below:
+  // - Foo foo()  // Direct application of |type_with_same_name_as_function|.
+  // - Foo* foo()  // |hasDescendant| traverses references/pointers.
+  // - RefPtr<Foo> foo()  // |hasDescendant| traverses template arguments.
+  auto type_containing_same_name_as_function =
+      qualType(anyOf(type_with_same_name_as_function,
+                     hasDescendant(type_with_same_name_as_function)));
+  // https://crbug.com/582312: Prepend "Get" if method name conflicts with
+  // return type.
+  auto conflict_matcher =
+      functionDecl(returns(type_containing_same_name_as_function));
+  if (!match(conflict_matcher, decl, context).empty())
     name = "Get" + name;
 
   return true;
 }
 
 bool GetNameForDecl(const clang::EnumConstantDecl& decl,
                     clang::ASTContext& context,
                     std::string& name) {
   StringRef original_name = decl.getName();
 
@@ skipping 619 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;
 }