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

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

Issue 360833002: Divorce PaintInvalidationState from LayoutState (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Address Julien's comments Created 6 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
« no previous file with comments | « Source/core/rendering/RenderView.h ('k') | Source/core/rendering/svg/RenderSVGBlock.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/rendering/RenderView.cpp
diff --git a/Source/core/rendering/RenderView.cpp b/Source/core/rendering/RenderView.cpp
index d98e8c0452a8ef9599544c0e56ee5c9faf5d5d30..56183865111d83e0753449d449593702ff86c56d 100644
--- a/Source/core/rendering/RenderView.cpp
+++ b/Source/core/rendering/RenderView.cpp
@@ -260,7 +260,7 @@ void RenderView::layout()
clearNeedsLayout();
}
-void RenderView::mapLocalToContainer(const RenderLayerModelObject* repaintContainer, TransformState& transformState, MapCoordinatesFlags mode, bool* wasFixed) const
+void RenderView::mapLocalToContainer(const RenderLayerModelObject* repaintContainer, TransformState& transformState, MapCoordinatesFlags mode, bool* wasFixed, const PaintInvalidationState* paintInvalidationState) const
{
ASSERT_UNUSED(wasFixed, !wasFixed || *wasFixed == static_cast<bool>(mode & IsFixed));
@@ -281,7 +281,7 @@ void RenderView::mapLocalToContainer(const RenderLayerModelObject* repaintContai
transformState.move(-frame()->view()->scrollOffset());
if (parentDocRenderer->isBox())
transformState.move(toLayoutSize(toRenderBox(parentDocRenderer)->contentBoxRect().location()));
- parentDocRenderer->mapLocalToContainer(repaintContainer, transformState, mode, wasFixed);
+ parentDocRenderer->mapLocalToContainer(repaintContainer, transformState, mode, wasFixed, paintInvalidationState);
return;
}
}
@@ -444,7 +444,7 @@ void RenderView::paintBoxDecorationBackground(PaintInfo& paintInfo, const Layout
}
}
-void RenderView::invalidateTreeAfterLayout(const RenderLayerModelObject& paintInvalidationContainer)
+void RenderView::invalidateTreeAfterLayout(const PaintInvalidationState& paintInvalidationState)
{
ASSERT(!needsLayout());
@@ -453,8 +453,7 @@ void RenderView::invalidateTreeAfterLayout(const RenderLayerModelObject& paintIn
if (doingFullRepaint() && !viewRect().isEmpty())
repaintViewRectangle(viewRect());
- LayoutState rootLayoutState(0, false, *this);
- RenderBlock::invalidateTreeAfterLayout(paintInvalidationContainer);
+ RenderBlock::invalidateTreeAfterLayout(paintInvalidationState);
}
void RenderView::repaintViewRectangle(const LayoutRect& repaintRect) const
@@ -496,7 +495,7 @@ void RenderView::repaintViewAndCompositedLayers()
compositor()->repaintCompositedLayers();
}
-void RenderView::mapRectToPaintInvalidationBacking(const RenderLayerModelObject* paintInvalidationContainer, LayoutRect& rect, bool fixed) const
+void RenderView::mapRectToPaintInvalidationBacking(const RenderLayerModelObject* paintInvalidationContainer, LayoutRect& rect, bool fixed, const PaintInvalidationState* paintInvalidationState) const
{
// If a container was specified, and was not 0 or the RenderView,
// then we should have found it by now.
« no previous file with comments | « Source/core/rendering/RenderView.h ('k') | Source/core/rendering/svg/RenderSVGBlock.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698