For SPv2, move updateAnimations() to follow scrolling and paint invalidation.

third_party/WebKit/Source/core/frame/FrameView.cpp

Index: third_party/WebKit/Source/core/frame/FrameView.cpp
diff --git a/third_party/WebKit/Source/core/frame/FrameView.cpp b/third_party/WebKit/Source/core/frame/FrameView.cpp
index 196f562691e386f9bdd7377e29038afb9aec1869..f79f35b263359fb6474db16ea5d694dba2d4c38b 100644
--- a/third_party/WebKit/Source/core/frame/FrameView.cpp
+++ b/third_party/WebKit/Source/core/frame/FrameView.cpp
@@ -2873,8 +2873,6 @@ void FrameView::updateLifecyclePhasesInternal(
       if (!RuntimeEnabledFeatures::slimmingPaintV2Enabled()) {
         view.compositor()->updateIfNeededRecursive();
       } else {
-        DocumentAnimations::updateAnimations(layoutView()->document());
-
         forAllNonThrottledFrameViews([](FrameView& frameView) {
           frameView.layoutView()->layer()->updateDescendantDependentFlags();
           frameView.layoutView()->commitPendingSelection();
@@ -2898,6 +2896,9 @@ void FrameView::updateLifecyclePhasesInternal(
 
         updateCompositedSelectionIfNeeded();
       }
+
+      if (RuntimeEnabledFeatures::slimmingPaintV2Enabled())
+        DocumentAnimations::updateAnimations(layoutView()->document());

[chrishtr, 2016/12/13 00:13:53]

It needs to be after updatePaintProperties, if animations will live on property
tree nodes.

[wkorman, 2016/12/15 20:14:04]

Indeed. Done.

     }
 
     if (targetState >= DocumentLifecycle::PrePaintClean) {