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

cc/animation/element_animations.h

 // 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.
 
 #ifndef CC_ANIMATION_ELEMENT_ANIMATIONS_H_
 #define CC_ANIMATION_ELEMENT_ANIMATIONS_H_
 
 #include <memory>
 #include <vector>
 
@@ skipping 42 matching lines @@
   void ClearAffectedElementTypes();
 
   void ElementRegistered(ElementId element_id, ElementListType list_type);
   void ElementUnregistered(ElementId element_id, ElementListType list_type);
 
   void AddPlayer(AnimationPlayer* player);
   void RemovePlayer(AnimationPlayer* player);
   bool IsEmpty() const;
 
   typedef base::ObserverList<AnimationPlayer> PlayersList;
-  PlayersList& players_list() const { return *players_list_.get(); }
+  const PlayersList& players_list() const { return players_list_; }
 
   // Ensures that the list of active animations on the main thread and the impl
   // thread are kept in sync. This function does not take ownership of the impl
   // thread ElementAnimations.
   void PushPropertiesTo(
       scoped_refptr<ElementAnimations> element_animations_impl);
 
   void Animate(base::TimeTicks monotonic_time);
 
   void UpdateState(bool start_ready_animations, AnimationEvents* events);
@@ skipping 118 matching lines @@
   void OnFilterAnimated(ElementListType list_type,
                         const FilterOperations& filters);
   void OnOpacityAnimated(ElementListType list_type, float opacity);
   void OnTransformAnimated(ElementListType list_type,
                            const gfx::Transform& transform);
   void OnScrollOffsetAnimated(ElementListType list_type,
                               const gfx::ScrollOffset& scroll_offset);
 
   static TargetProperties GetPropertiesMaskForAnimationState();
 
-  std::unique_ptr<PlayersList> players_list_;
+  PlayersList players_list_;
   AnimationHost* animation_host_;
   ElementId element_id_;
 
   // This is used to ensure that we don't spam the animation host.
   bool is_active_;
 
   base::TimeTicks last_tick_time_;
 
   bool has_element_in_active_list_;
   bool has_element_in_pending_list_;
 
   bool scroll_offset_animation_was_interrupted_;
 
   bool needs_push_properties_;
 
   PropertyAnimationState active_state_;
   PropertyAnimationState pending_state_;
 
   bool needs_update_impl_client_state_;
 
   DISALLOW_COPY_AND_ASSIGN(ElementAnimations);
 };
 
 }  // namespace cc
 
 #endif  // CC_ANIMATION_ELEMENT_ANIMATIONS_H_