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

Unified Diff: Source/core/layout/LayoutFullScreen.cpp

Issue 1025213002: Begin tracking why layout is invalidated (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 9 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/layout/LayoutFullScreen.cpp
diff --git a/Source/core/layout/LayoutFullScreen.cpp b/Source/core/layout/LayoutFullScreen.cpp
index f05f0f0d285eb291e86c65a7a230d8aa0e4b16d6..0fce200449a1ecdfba9546fdd28cb198803e7854 100644
--- a/Source/core/layout/LayoutFullScreen.cpp
+++ b/Source/core/layout/LayoutFullScreen.cpp
@@ -148,11 +148,11 @@ LayoutObject* LayoutFullScreen::wrapRenderer(LayoutObject* object, LayoutObject*
// Always just do a full layout to ensure that line boxes get deleted properly.
// Because objects moved from |parent| to |fullscreenRenderer|, we want to
// make new line boxes instead of leaving the old ones around.
- parent->setNeedsLayoutAndPrefWidthsRecalcAndFullPaintInvalidation();
- containingBlock->setNeedsLayoutAndPrefWidthsRecalcAndFullPaintInvalidation();
+ parent->setNeedsLayoutAndPrefWidthsRecalcAndFullPaintInvalidation(LayoutInvalidationReason::Fullscreen);
+ containingBlock->setNeedsLayoutAndPrefWidthsRecalcAndFullPaintInvalidation(LayoutInvalidationReason::Fullscreen);
}
fullscreenRenderer->addChild(object);
- fullscreenRenderer->setNeedsLayoutAndPrefWidthsRecalcAndFullPaintInvalidation();
+ fullscreenRenderer->setNeedsLayoutAndPrefWidthsRecalcAndFullPaintInvalidation(LayoutInvalidationReason::Fullscreen);
}
ASSERT(document);
@@ -175,7 +175,7 @@ void LayoutFullScreen::unwrapRenderer()
toLayoutBox(child)->clearOverrideSize();
child->remove();
parent()->addChild(child, this);
- parent()->setNeedsLayoutAndPrefWidthsRecalcAndFullPaintInvalidation();
+ parent()->setNeedsLayoutAndPrefWidthsRecalcAndFullPaintInvalidation(LayoutInvalidationReason::Fullscreen);
}
}
if (placeholder())
@@ -201,7 +201,7 @@ void LayoutFullScreen::createPlaceholder(PassRefPtr<LayoutStyle> style, const La
m_placeholder->setStyle(style);
if (parent()) {
parent()->addChild(m_placeholder, this);
- parent()->setNeedsLayoutAndPrefWidthsRecalcAndFullPaintInvalidation();
+ parent()->setNeedsLayoutAndPrefWidthsRecalcAndFullPaintInvalidation(LayoutInvalidationReason::Fullscreen);
}
} else {
m_placeholder->setStyle(style);

Powered by Google App Engine
This is Rietveld 408576698