CC Animation: Rework iterations over players to use the range-based for loop.

cc/animation/element_animations_unittest.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.
 
 #include "cc/animation/element_animations.h"
 
 #include "cc/animation/animation_delegate.h"
 #include "cc/animation/animation_events.h"
 #include "cc/animation/animation_host.h"
 #include "cc/animation/animation_id_provider.h"
@@ skipping 157 matching lines @@
   EXPECT_EQ(element_animations, player2->element_animations());
 
   PushProperties();
   GetImplTimelineAndPlayerByID();
 
   scoped_refptr<ElementAnimations> element_animations_impl =
       player_impl_->element_animations();
   EXPECT_TRUE(element_animations_impl);
 
   int list_size_before = 0;
-  ElementAnimations::PlayersList::Iterator it(
-      &element_animations_impl_->players_list());
-  AnimationPlayer* player;
-  while ((player = it.GetNext()) != nullptr) {
-    EXPECT_TRUE(timeline_->GetPlayerById(player->id()));
+  for (auto& player : element_animations_impl_->players_list()) {
+    EXPECT_TRUE(timeline_->GetPlayerById(player.id()));
     ++list_size_before;
   }
   EXPECT_EQ(3, list_size_before);
 
   player2->DetachElement();
   EXPECT_FALSE(player2->element_animations());
   EXPECT_EQ(element_animations, player_->element_animations());
   EXPECT_EQ(element_animations, player1->element_animations());
 
   PushProperties();
   EXPECT_EQ(element_animations_impl, player_impl_->element_animations());
 
   int list_size_after = 0;
-  it = ElementAnimations::PlayersList::Iterator(
-      &element_animations_impl_->players_list());
-  while ((player = it.GetNext()) != nullptr) {
-    EXPECT_TRUE(timeline_->GetPlayerById(player->id()));
+  for (auto& player : element_animations_impl_->players_list()) {
+    EXPECT_TRUE(timeline_->GetPlayerById(player.id()));
     ++list_size_after;
   }
   EXPECT_EQ(2, list_size_after);
 }
 
 TEST_F(ElementAnimationsTest, SyncNewAnimation) {
   CreateTestLayer(true, false);
   AttachTimelinePlayerLayer();
   CreateImplTimelineAndPlayer();
 
@@ skipping 3417 matching lines @@
   EXPECT_FALSE(player_->IsPotentiallyAnimatingProperty(
       TargetProperty::OPACITY, ElementListType::ACTIVE));
   EXPECT_FALSE(player_->IsCurrentlyAnimatingProperty(TargetProperty::OPACITY,
                                                      ElementListType::PENDING));
   EXPECT_FALSE(player_->IsCurrentlyAnimatingProperty(TargetProperty::OPACITY,
                                                      ElementListType::ACTIVE));
 }
 
 }  // namespace
 }  // namespace cc