Revert "CC Animation: Introduce some dirty flags to optimize PushProperties on commit"

cc/animation/animation_timeline.cc

Index: cc/animation/animation_timeline.cc
diff --git a/cc/animation/animation_timeline.cc b/cc/animation/animation_timeline.cc
index d76e9821a5e26b35256ce2b2663b69e5a27deabf..5b2b10653f03bc27a1682b0892057291220ceb7d 100644
--- a/cc/animation/animation_timeline.cc
+++ b/cc/animation/animation_timeline.cc
@@ -6,7 +6,6 @@
 
 #include <algorithm>
 
-#include "cc/animation/animation_host.h"
 #include "cc/animation/animation_player.h"
 
 namespace cc {
@@ -16,10 +15,8 @@ scoped_refptr<AnimationTimeline> AnimationTimeline::Create(int id) {
 }
 
 AnimationTimeline::AnimationTimeline(int id)
-    : id_(id),
-      animation_host_(),
-      needs_push_properties_(false),
-      is_impl_only_(false) {}
+    : id_(id), animation_host_(), is_impl_only_(false) {
+}
 
 AnimationTimeline::~AnimationTimeline() {
   for (auto& kv : id_to_player_map_)
@@ -32,14 +29,9 @@ scoped_refptr<AnimationTimeline> AnimationTimeline::CreateImplInstance() const {
 }
 
 void AnimationTimeline::SetAnimationHost(AnimationHost* animation_host) {
-  if (animation_host_ == animation_host)
-    return;
-
   animation_host_ = animation_host;
   for (auto& kv : id_to_player_map_)
     kv.second->SetAnimationHost(animation_host);
-
-  SetNeedsPushProperties();
 }
 
 void AnimationTimeline::AttachPlayer(scoped_refptr<AnimationPlayer> player) {
@@ -47,16 +39,12 @@ void AnimationTimeline::AttachPlayer(scoped_refptr<AnimationPlayer> player) {
   player->SetAnimationHost(animation_host_);
   player->SetAnimationTimeline(this);
   id_to_player_map_.insert(std::make_pair(player->id(), std::move(player)));
-
-  SetNeedsPushProperties();
 }
 
 void AnimationTimeline::DetachPlayer(scoped_refptr<AnimationPlayer> player) {
   DCHECK(player->id());
   ErasePlayer(player);
   id_to_player_map_.erase(player->id());
-
-  SetNeedsPushProperties();
 }
 
 AnimationPlayer* AnimationTimeline::GetPlayerById(int player_id) const {
@@ -68,23 +56,12 @@ void AnimationTimeline::ClearPlayers() {
   for (auto& kv : id_to_player_map_)
     ErasePlayer(kv.second);
   id_to_player_map_.clear();
-
-  SetNeedsPushProperties();
-}
-
-void AnimationTimeline::SetNeedsPushProperties() {
-  needs_push_properties_ = true;
-  if (animation_host_)
-    animation_host_->SetNeedsPushProperties();
 }
 
 void AnimationTimeline::PushPropertiesTo(AnimationTimeline* timeline_impl) {
-  if (needs_push_properties_) {
-    needs_push_properties_ = false;
-    PushAttachedPlayersToImplThread(timeline_impl);
-    RemoveDetachedPlayersFromImplThread(timeline_impl);
-    PushPropertiesToImplThread(timeline_impl);
-  }
+  PushAttachedPlayersToImplThread(timeline_impl);
+  RemoveDetachedPlayersFromImplThread(timeline_impl);
+  PushPropertiesToImplThread(timeline_impl);
 }
 
 void AnimationTimeline::PushAttachedPlayersToImplThread(
@@ -126,11 +103,9 @@ void AnimationTimeline::PushPropertiesToImplThread(
     AnimationTimeline* timeline_impl) {
   for (auto& kv : id_to_player_map_) {
     AnimationPlayer* player = kv.second.get();
-    if (player->needs_push_properties()) {
-      AnimationPlayer* player_impl = timeline_impl->GetPlayerById(player->id());
-      if (player_impl)
-        player->PushPropertiesTo(player_impl);
-    }
+    AnimationPlayer* player_impl = timeline_impl->GetPlayerById(player->id());
+    if (player_impl)
+      player->PushPropertiesTo(player_impl);
   }
 }