Remove base::ObserverList<T>::Iter::GetNext().

base/observer_list_unittest.cc

 // Copyright (c) 2012 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 "base/observer_list.h"
 #include "base/observer_list_threadsafe.h"
 
 #include <vector>
 
 #include "base/compiler_specific.h"
@@ skipping 887 matching lines @@
 TEST(ObserverListTest, AddObserverInTheLastObserve) {
   using FooList = ObserverList<Foo>;
   FooList observer_list;
 
   AddInObserve<FooList> a(&observer_list);
   Adder b(-1);
 
   a.SetToAdd(&b);
   observer_list.AddObserver(&a);
 
-  FooList::Iterator it(&observer_list);
-  Foo* foo;
-  while ((foo = it.GetNext()) != nullptr)
-    foo->Observe(10);
+  auto it = observer_list.begin();
+  while (it != observer_list.end()) {
+    auto& observer = *it;
+    ++it;

[danakj, 2016/10/14 21:52:10]

Leave a comment explaining why youre doing this ++ before the Observe

+    observer.Observe(10);
+  }
 
   EXPECT_EQ(-10, b.total);
 }
 
 }  // namespace base