Skipping renaming of begin/end/etc. should be unconditional.

tools/clang/rewrite_to_chrome_style/tests/template-original.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 240 matching lines @@
   return operator T*();
 }
 
 class Class {
  public:
   virtual void virtualMethodInBlink() {}
 };
 
 }  // namespace cxx_dependent_scope_member_expr_testing
 
+namespace blacklisting_of_renaming_of_begin_method {
+
+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 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