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

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

Issue 18601002: Add infrastructure for partial layouts (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: If text autosizing is enabled, only partial layout for the second of two layouts 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/RenderObject.cpp
diff --git a/Source/core/rendering/RenderObject.cpp b/Source/core/rendering/RenderObject.cpp
index df6b2c140ba600fa9daad7b01964cdd1e031094e..33c16c21ef66f5ed2be9b3a44d59b72a3e2e6e6f 100644
--- a/Source/core/rendering/RenderObject.cpp
+++ b/Source/core/rendering/RenderObject.cpp
@@ -746,6 +746,16 @@ void RenderObject::checkBlockPositionedObjectsNeedLayout()
if (isRenderBlock())
toRenderBlock(this)->checkPositionedObjectsNeedLayout();
}
+
+void RenderObject::checkNotInPartialLayout()
+{
+ if (frame()) {
esprehn 2013/08/18 03:43:15 Add a frameView(), also I don't know how this can
pdr. 2013/08/20 06:19:10 Done. I changed this to not even check for null s
+ if (FrameView* view = frame()->view()) {
+ // During partial layout, setNeedsLayout(true or false) should not be called.
+ ASSERT(!view->shouldStopPartialLayout());
+ }
+ }
+}
#endif
void RenderObject::setPreferredLogicalWidthsDirty(bool shouldBeDirty, MarkingBehavior markParents)

Powered by Google App Engine
This is Rietveld 408576698