ElementAnimations should hold an ObservableList of AnimationPlayers.

cc/animation/element_animations.h

Index: cc/animation/element_animations.h
diff --git a/cc/animation/element_animations.h b/cc/animation/element_animations.h
index a46560450b27038a547b6c5ded9651837769ef14..b5aa34ff9d2b16e59bda7bede656f8537ea4569a 100644
--- a/cc/animation/element_animations.h
+++ b/cc/animation/element_animations.h
@@ -12,6 +12,7 @@
 #include "base/containers/linked_list.h"

[loyso (OOO), 2016/08/05 00:13:25]

This is not needed anymore.

[ymalik, 2016/08/05 00:54:04]

Done.

 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
+#include "base/observer_list.h"
 #include "base/time/time.h"
 #include "cc/animation/animation.h"
 #include "cc/animation/animation_curve.h"
@@ -62,9 +63,8 @@ class CC_EXPORT ElementAnimations : public base::RefCounted<ElementAnimations> {
   void RemovePlayer(AnimationPlayer* player);
   bool IsEmpty() const;
 
-  typedef base::LinkedList<AnimationPlayer> PlayersList;
-  typedef base::LinkNode<AnimationPlayer> PlayersListNode;
-  const PlayersList& players_list() const { return *players_list_.get(); }
+  typedef base::ObserverList<AnimationPlayer> PlayersList;
+  PlayersList& players_list() const { return *players_list_.get(); }
 
   // 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