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

Unified Diff: Source/core/frame/FrameView.cpp

Issue 280633002: Rename RenderObject::setNeedsLayout to setNeedsLayoutAndFullRepaint (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebase Created 6 years, 7 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
« no previous file with comments | « Source/core/dom/Element.cpp ('k') | Source/core/html/HTMLFrameSetElement.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/frame/FrameView.cpp
diff --git a/Source/core/frame/FrameView.cpp b/Source/core/frame/FrameView.cpp
index 43563075d6a153f398b270309a3b0694f28f68a1..9eee49932edd4e058f125b34ea1196fc5e63875e 100644
--- a/Source/core/frame/FrameView.cpp
+++ b/Source/core/frame/FrameView.cpp
@@ -739,7 +739,7 @@ inline void FrameView::forceLayoutParentViewIfNeeded()
RefPtr<FrameView> frameView = ownerRenderer->frame()->view();
// Mark the owner renderer as needing layout.
- ownerRenderer->setNeedsLayoutAndPrefWidthsRecalc();
+ ownerRenderer->setNeedsLayoutAndPrefWidthsRecalcAndFullRepaint();
// Synchronously enter layout, to layout the view containing the host object/embed/iframe.
ASSERT(frameView);
@@ -1309,13 +1309,13 @@ void FrameView::viewportConstrainedVisibleContentSizeChanged(bool widthChanged,
if (style->width().isFixed() && (style->left().isAuto() || style->right().isAuto()))
renderer->setNeedsPositionedMovementLayout();
else
- renderer->setNeedsLayout();
+ renderer->setNeedsLayoutAndFullRepaint();
}
if (heightChanged) {
if (style->height().isFixed() && (style->top().isAuto() || style->bottom().isAuto()))
renderer->setNeedsPositionedMovementLayout();
else
- renderer->setNeedsLayout();
+ renderer->setNeedsLayoutAndFullRepaint();
}
}
}
@@ -1928,7 +1928,7 @@ bool FrameView::needsLayout() const
void FrameView::setNeedsLayout()
{
if (RenderView* renderView = this->renderView())
- renderView->setNeedsLayout();
+ renderView->setNeedsLayoutAndFullRepaint();
}
bool FrameView::isTransparent() const
@@ -2941,7 +2941,7 @@ void FrameView::forceLayoutForPagination(const FloatSize& pageSize, const FloatS
LayoutUnit flooredPageLogicalHeight = static_cast<LayoutUnit>(pageLogicalHeight);
renderView->setLogicalWidth(flooredPageLogicalWidth);
renderView->setPageLogicalHeight(flooredPageLogicalHeight);
- renderView->setNeedsLayoutAndPrefWidthsRecalc();
+ renderView->setNeedsLayoutAndPrefWidthsRecalcAndFullRepaint();
forceLayout();
// If we don't fit in the given page width, we'll lay out again. If we don't fit in the
@@ -2961,7 +2961,7 @@ void FrameView::forceLayoutForPagination(const FloatSize& pageSize, const FloatS
flooredPageLogicalHeight = static_cast<LayoutUnit>(pageLogicalHeight);
renderView->setLogicalWidth(flooredPageLogicalWidth);
renderView->setPageLogicalHeight(flooredPageLogicalHeight);
- renderView->setNeedsLayoutAndPrefWidthsRecalc();
+ renderView->setNeedsLayoutAndPrefWidthsRecalcAndFullRepaint();
forceLayout();
const LayoutRect& updatedDocumentRect = renderView->documentRect();
« no previous file with comments | « Source/core/dom/Element.cpp ('k') | Source/core/html/HTMLFrameSetElement.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698