Throttle rendering pipeline for invisible iframes

Source/core/page/PageAnimator.cpp

Index: Source/core/page/PageAnimator.cpp
diff --git a/Source/core/page/PageAnimator.cpp b/Source/core/page/PageAnimator.cpp
index 93b0e21b80b05b9b7dfee899b61685aae28cf4b0..788153a4a6b3a87c6ad5f5e4d79d9808b59e5d06 100644
--- a/Source/core/page/PageAnimator.cpp
+++ b/Source/core/page/PageAnimator.cpp
@@ -39,8 +39,13 @@ void PageAnimator::serviceScriptedAnimations(double monotonicAnimationStartTime)
 
     WillBeHeapVector<RefPtrWillBeMember<Document>> documents;
     for (Frame* frame = m_page->mainFrame(); frame; frame = frame->tree().traverseNext()) {
-        if (frame->isLocalFrame())
-            documents.append(toLocalFrame(frame)->document());
+        if (!frame->isLocalFrame())
+            continue;
+        LocalFrame* localFrame = toLocalFrame(frame);
+        // FIXME: Keep the animation timing ticking while throttled.

[esprehn, 2015/09/02 21:10:37]

I don't think you need to do anything special here, the animation system should
be able to do an on demand update?

[Sami, 2015/09/03 17:23:20]

This is referring to dstockwell@'s earlier comment about making sure the
animation time remains valid if someone queries for it
(DocumentAnimations::updateAnimationTimingForAnimationFrame) but still avoiding
any other animation related processing.

+        if (localFrame->shouldThrottleRenderingPipeline())
+            continue;
+        documents.append(localFrame->document());
     }
 
     for (auto& document : documents) {
@@ -91,7 +96,7 @@ void PageAnimator::updateLayoutAndStyleForPainting(LocalFrame* rootFrame)
 
     // setFrameRect may have the side-effect of causing existing page layout to
     // be invalidated, so layout needs to be called last.
-    view->updateAllLifecyclePhases();
+    view->updateAllLifecyclePhases(FrameView::LifecycleThrottlingMode::Allow);

[esprehn, 2015/09/02 21:10:37]

Lets un-nest the enum.

[Sami, 2015/09/03 17:23:20]

Done.

 }
 
 }