Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1258)

Unified Diff: Source/core/rendering/RenderLayerCompositor.cpp

Issue 135693003: Defer starting of animations until after compositing update (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Move some AnimationClock functions to cpp. Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/core/rendering/RenderLayerCompositor.cpp
diff --git a/Source/core/rendering/RenderLayerCompositor.cpp b/Source/core/rendering/RenderLayerCompositor.cpp
index 4ea971ba77e2ad55d00844322edf4f4cfe838745..5900e0af1761ee74759fe79f0e102c1819fc6a3f 100644
--- a/Source/core/rendering/RenderLayerCompositor.cpp
+++ b/Source/core/rendering/RenderLayerCompositor.cpp
@@ -401,8 +401,17 @@ void RenderLayerCompositor::updateCompositingLayers()
return;
lifecycle().advanceTo(DocumentLifecycle::InCompositingUpdate);
- DocumentLifecycle::Scope lifecycleScope(lifecycle(), DocumentLifecycle::CompositingClean);
+ updateCompositingLayersInternal();
+
+ lifecycle().advanceTo(DocumentLifecycle::CompositingClean);
+
+ DocumentAnimations::startPendingAnimations(m_renderView->document());
+ DocumentAnimations::dispatchAnimationEventsAsync(m_renderView->document());
esprehn 2014/02/24 23:14:35 You should ASSERT() at the end that the lifecycle(
dstockwell 2014/02/25 00:52:03 Done.
+}
+
+void RenderLayerCompositor::updateCompositingLayersInternal()
+{
if (isMainFrame() && m_renderView->frameView())
finishCompositingUpdateForFrameTree(&m_renderView->frameView()->frame());

Powered by Google App Engine
This is Rietveld 408576698