Throttle rendering pipeline for invisible frames

third_party/WebKit/Source/core/page/PageAnimator.cpp

Index: third_party/WebKit/Source/core/page/PageAnimator.cpp
diff --git a/third_party/WebKit/Source/core/page/PageAnimator.cpp b/third_party/WebKit/Source/core/page/PageAnimator.cpp
index 93b0e21b80b05b9b7dfee899b61685aae28cf4b0..629153a40aa0b9e83a7b884870b4ef9914169b5b 100644
--- a/third_party/WebKit/Source/core/page/PageAnimator.cpp
+++ b/third_party/WebKit/Source/core/page/PageAnimator.cpp
@@ -44,7 +44,10 @@ void PageAnimator::serviceScriptedAnimations(double monotonicAnimationStartTime)
     }
 
     for (auto& document : documents) {
+        DocumentAnimations::updateAnimationTimingForAnimationFrame(*document, monotonicAnimationStartTime);
         if (document->view()) {
+            if (document->view()->shouldThrottleRendering())
+                continue;
             document->view()->scrollableArea()->serviceScrollAnimations(monotonicAnimationStartTime);
 
             if (const FrameView::ScrollableAreaSet* animatingScrollableAreas = document->view()->animatingScrollableAreas()) {
@@ -56,7 +59,7 @@ void PageAnimator::serviceScriptedAnimations(double monotonicAnimationStartTime)
                     scrollableArea->serviceScrollAnimations(monotonicAnimationStartTime);
             }
         }
-        DocumentAnimations::updateAnimationTimingForAnimationFrame(*document, monotonicAnimationStartTime);
+        // TODO(skyostil): Find out if these functions need to be run for documents without views.

[esprehn, 2015/10/16 21:40:45]

They definitely shouldn't run for documents without views, that's a bug.

[Sami, 2015/10/19 10:51:06]

Right, I wasn't sure if we were sure about that. Updated the text.

         SVGDocumentExtensions::serviceOnAnimationFrame(*document, monotonicAnimationStartTime);
         document->serviceScriptedAnimations(monotonicAnimationStartTime);
     }