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

tools/clang/rewrite_to_chrome_style/tests/functions-original.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 {
 
 // Tests that the prototype for a function is updated.
 int testFunctionThatTakesTwoInts(int x, int y);
 
 // Test that the actual function definition is also updated.
 int testFunctionThatTakesTwoInts(int x, int y) {
   if (x == 0)
     return y;
   // Calls to the function also need to be updated.
   return testFunctionThatTakesTwoInts(x - 1, y + 1);
 }
 
+// This is named like the begin() method which isn't renamed, but
+// here it's not a method so it should be.
+void begin() {}

[dcheng, 2016/01/28 06:58:12]

Hopefully there are no uses of non-member begin() / end() in Blink.

[danakj, 2016/01/28 19:47:58]

Ya, but better safe than sorry? :)

+
 // Note: F is already Google style and should not change.
 void F() {
   // Test referencing a function without calling it.
   int (*functionPointer)(int, int) = &testFunctionThatTakesTwoInts;
 }
 
 }  // namespace blink
 
 void G() {
   blink::testFunctionThatTakesTwoInts(1, 2);
 }