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

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

Issue 301843002: Store repaint rects in the coordinate space of their backing GraphicsLayer. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Changed name. 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/rendering/RenderBox.h ('k') | Source/core/rendering/RenderFrameSet.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/rendering/RenderBox.cpp
diff --git a/Source/core/rendering/RenderBox.cpp b/Source/core/rendering/RenderBox.cpp
index 728025e47a39d2c27a2bbd96138be15a2b6af153..f901225e76b869181d9d9afd317263ac9f810de0 100644
--- a/Source/core/rendering/RenderBox.cpp
+++ b/Source/core/rendering/RenderBox.cpp
@@ -1582,7 +1582,7 @@ void RenderBox::repaintTreeAfterLayout(const RenderLayerModelObject& repaintCont
const LayoutRect oldRepaintRect = previousRepaintRect();
const LayoutPoint oldPositionFromRepaintContainer = previousPositionFromRepaintContainer();
- setPreviousRepaintRect(clippedOverflowRectForRepaint(&newRepaintContainer));
+ setPreviousRepaintRect(boundsRectForRepaint(&newRepaintContainer));
setPreviousPositionFromRepaintContainer(positionFromRepaintContainer(&newRepaintContainer));
// If we are set to do a full repaint that means the RenderView will be
@@ -2011,11 +2011,11 @@ LayoutRect RenderBox::clippedOverflowRectForRepaint(const RenderLayerModelObject
r.move(v->layoutDelta());
}
- computeRectForRepaint(repaintContainer, r);
+ mapRectToRepaintBacking(repaintContainer, r);
return r;
}
-void RenderBox::computeRectForRepaint(const RenderLayerModelObject* repaintContainer, LayoutRect& rect, bool fixed) const
+void RenderBox::mapRectToRepaintBacking(const RenderLayerModelObject* repaintContainer, LayoutRect& rect, bool fixed) const
{
// The rect we compute at each step is shifted by our x/y offset in the parent container's coordinate space.
// Only when we cross a writing mode boundary will we have to possibly flipForWritingMode (to convert into a more appropriate
@@ -2112,7 +2112,7 @@ void RenderBox::computeRectForRepaint(const RenderLayerModelObject* repaintConta
return;
}
- o->computeRectForRepaint(repaintContainer, rect, fixed);
+ o->mapRectToRepaintBacking(repaintContainer, rect, fixed);
}
void RenderBox::repaintDuringLayoutIfMoved(const LayoutRect& oldRect)
« no previous file with comments | « Source/core/rendering/RenderBox.h ('k') | Source/core/rendering/RenderFrameSet.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698