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

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

Issue 18601002: Add infrastructure for partial layouts (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Address reviewer comments. Add PartialLayoutState and PartialLayoutDisabler Created 7 years, 4 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/SubtreeLayoutScope.cpp
diff --git a/Source/core/rendering/SubtreeLayoutScope.cpp b/Source/core/rendering/SubtreeLayoutScope.cpp
index c3bfefe745a64383618009fb6e6b9e43b7744e81..cec1e3356878614c732e1e90015f8aed60a600cf 100644
--- a/Source/core/rendering/SubtreeLayoutScope.cpp
+++ b/Source/core/rendering/SubtreeLayoutScope.cpp
@@ -44,6 +44,12 @@ SubtreeLayoutScope::SubtreeLayoutScope(RenderObject* root)
SubtreeLayoutScope::~SubtreeLayoutScope()
{
+ // Partial layout early-exits layotu and will leave the tree as needing layout.
ojan 2013/08/27 01:33:41 typo: layotu
+ if (m_root->frameView()->partialLayout().shouldStop()) {
+ ASSERT(m_root->needsLayout());
+ return;
+ }
+
RELEASE_ASSERT(!m_root->needsLayout());
#ifndef NDEBUG

Powered by Google App Engine
This is Rietveld 408576698