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

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

Index: tools/clang/rewrite_to_chrome_style/tests/methods-original.cc
diff --git a/tools/clang/rewrite_to_chrome_style/tests/methods-original.cc b/tools/clang/rewrite_to_chrome_style/tests/methods-original.cc
index bc9255bfaa296e37073972a00624430322fd0900..1f8f3a002e870b080c8b0d3ecd434045adb1ac8f 100644
--- a/tools/clang/rewrite_to_chrome_style/tests/methods-original.cc
+++ b/tools/clang/rewrite_to_chrome_style/tests/methods-original.cc
@@ -74,14 +74,6 @@ class Other {
   static void lock() {}
 };
 
-class NonIterators {
-  // begin()/end() and friends are renamed if they don't return an iterator.
-  void begin() {}
-  int end() { return 0; }
-  void rbegin() {}
-  int rend() { return 0; }
-};
-
 // Test that the actual method definition is also updated.
 void Task::doTheWork() {
   reallyDoTheWork();