add has_element_in_any_list check in ElementAnimations::UpdateActivation

cc/animation/element_animations.cc

Index: cc/animation/element_animations.cc
diff --git a/cc/animation/element_animations.cc b/cc/animation/element_animations.cc
index f8a7e9e9bef15e53dfa6504c74bc2f24867d0169..f2e2aa2d6a460f834d91f9971533f1447f5c021c 100644
--- a/cc/animation/element_animations.cc
+++ b/cc/animation/element_animations.cc
@@ -373,7 +373,7 @@ void ElementAnimations::UpdateActivation(ActivationType type) {
       }
     }
 
-    if (is_active_ && (!was_active || force)) {
+    if (is_active_ && (!was_active || force) && has_element_in_any_list()) {

[loyso (OOO), 2016/11/28 04:27:04]

Sohuldn't it be simple with a semantics similar to:
if (!has_element_in_any_list()) {
animation_host_->DidDeactivateElementAnimations(this); return; }
at the beginning of the function?

[junchao.han, 2016/11/28 07:27:45]

for 2 unit tests failed in patch set 3, UpdateActivation(ActivationType::FORCE)
is called. So the corner case only apply to (is_active_ && !was_active &&
has_element_in_any_list()). A new patch has been uploaded. PTAL

       animation_host_->DidActivateElementAnimations(this);
     } else if (!is_active_ && (was_active || force)) {
       // Resetting last_tick_time_ here ensures that calling ::UpdateState