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

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

Index: tools/clang/rewrite_to_chrome_style/tests/methods-expected.cc
diff --git a/tools/clang/rewrite_to_chrome_style/tests/methods-expected.cc b/tools/clang/rewrite_to_chrome_style/tests/methods-expected.cc
index 569e446cd2c7c54a200adfafaffef2f733c38844..1eff05bbbb1e633f2c32096aac2ba21eba459da1 100644
--- a/tools/clang/rewrite_to_chrome_style/tests/methods-expected.cc
+++ b/tools/clang/rewrite_to_chrome_style/tests/methods-expected.cc
@@ -70,14 +70,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();