rewrite_to_chrome_style: Don't rename begin()/end()/rbegin()/rend().

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

 // 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.
 
 namespace blink {
 
 class Task {
  public:
   // Already style-compliant methods shouldn't change.
   void OutputDebugString() {}
 
   // Tests that the declarations for methods are updated.
   void DoTheWork();
   virtual void ReallyDoTheWork() = 0;
 
   // Note: this is purposely copyable and assignable, to make sure the Clang
   // tool doesn't try to emit replacements for things that aren't explicitly
   // written.
 
   // Overloaded operators should not be rewritten.
   Task& operator++() { return *this; }
 
   // Conversion functions should not be rewritten.
   explicit operator int() const { return 42; }
+
+  // These are special functions that we don't rename so that range-based
+  // for loops and STL things work.
+  void begin() {}
+  void end() {}
+  void rbegin() {}
+  void rend() {}
+  // The trace() method is used by Oilpan, we shouldn't rename it.
+  void trace() {}
+};
+
+class Other {
+  // Static begin/end/trace don't count, and should be renamed.
+  static void Begin() {}
+  static void End() {}
+  static void Trace() {}
 };
 
 // Test that the actual method definition is also updated.
 void Task::DoTheWork() {
   ReallyDoTheWork();
 }
 
 }  // namespace blink
 
 namespace Moo {
@@ skipping 12 matching lines @@
 
 // Finally, test that method pointers are also updated.
 void F() {
   void (blink::Task::*p1)() = &blink::Task::DoTheWork;
   void (blink::Task::*p2)() = &BovineTask::DoTheWork;
   void (blink::Task::*p3)() = &blink::Task::ReallyDoTheWork;
   void (BovineTask::*p4)() = &BovineTask::ReallyDoTheWork;
 }
 
 }  // namespace Moo