Look at canonical decl, when deciding whether to rename or not.

tools/clang/rewrite_to_chrome_style/tests/template-expected.cc

 // Copyright 2016 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.
 
 #include <type_traits>
 
 namespace not_blink {
 
 void function(int x) {}
 
@@ skipping 320 matching lines @@
 
 template <typename T>
 class IntrusiveHeap {
  public:
   // https://crbug.com/672353: |begin| shouldn't be rewritten to |Begin|.
   const T* begin() const { return nullptr; }
 };
 
 }  // namespace blacklisting_of_renaming_of_begin_method
 
+namespace specializations {
+
+template <unsigned long sizeOfValue>
+int ToV8SignedIntegerInternal(long value);
+
+template <>
+int ToV8SignedIntegerInternal<4>(long value) {
+  return 123 + value;
+}
+
+template <>
+int ToV8SignedIntegerInternal<8>(long value) {
+  return 456 + value;
+}
+
+int ToV8(int value) {
+  return ToV8SignedIntegerInternal<sizeof value>(value);
+}
+
+int ToV8(long value) {
+  return ToV8SignedIntegerInternal<sizeof value>(value);
+}
+
+}  // namespace specializations
+

[Łukasz Anforowicz, 2017/01/21 01:04:01]

These tests are somewhat unrelated to the main changes in the CL, but I managed
to break these tests with the first patchset - see https://crbug.com/682835#c2
for details.

 }  // namespace blink
 
 namespace not_blink {
 
 namespace cxx_dependent_scope_member_expr_testing {
 
 class Base : public ::blink::cxx_dependent_scope_member_expr_testing::Class {
  public:
   virtual void virtualMethod() {}
 };
@@ skipping 20 matching lines @@
     // Need to rewrite |f| to |F| below (because this method name
     // does get rewritten when processing blink::test_unnamed_arg::Class).
     // See also https://crbug.com/670434.
     output->F(123);
   }
 };
 
 }  // namespace blink_methods_called_from_mojo_traits_are_not_rewritten
 
 }  // namespace not_blink