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

Unified Diff: Source/core/dom/Document.cpp

Issue 19782002: Make Blink stop scheduling its own Layout, and use the compositor's timer instead (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: With test changes Created 7 years, 5 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/dom/Document.cpp
diff --git a/Source/core/dom/Document.cpp b/Source/core/dom/Document.cpp
index b3fc7f14359dada020f4dfa5ed5727064c2bfc11..fa014066699bfd51c1b31e6084fadf777b97c76c 100644
--- a/Source/core/dom/Document.cpp
+++ b/Source/core/dom/Document.cpp
@@ -1718,7 +1718,7 @@ void Document::updateLayout()
StackStats::LayoutCheckPoint layoutCheckPoint;
// Only do a layout if changes have occurred that make it necessary.
- if (frameView && renderer() && (frameView->layoutPending() || renderer()->needsLayout()))
+ if (frameView && renderer() && (frameView->layoutIsScheduledWithEmbedder() || renderer()->needsLayout()))
frameView->layout();
// FIXME: Using a Task doesn't look a good idea.
@@ -2316,9 +2316,10 @@ bool Document::shouldScheduleLayout()
bool Document::shouldParserYieldAgressivelyBeforeScriptExecution()
{
- return view() && view()->layoutPending() && !minimumLayoutDelay();
+ return view() && view()->layoutIsScheduledWithEmbedder() && !minimumLayoutDelay();
}
+// FIXME: I believe this whole function is useless.
int Document::minimumLayoutDelay()
{
if (m_overMinimumLayoutThreshold)
« no previous file with comments | « LayoutTests/platform/linux/svg/as-image/animated-svg-as-image-same-image-expected.png ('k') | Source/core/page/FrameView.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698