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

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

Issue 1428643004: Repaint on interest rect change (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@EnableSyncPaint
Patch Set: Created 5 years, 2 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: third_party/WebKit/Source/core/frame/FrameView.cpp
diff --git a/third_party/WebKit/Source/core/frame/FrameView.cpp b/third_party/WebKit/Source/core/frame/FrameView.cpp
index 8b43865107c3c03ed3f4aef590f6580215115b9a..02b447a259761d68b7187bcecd96f8276c9523f6 100644
--- a/third_party/WebKit/Source/core/frame/FrameView.cpp
+++ b/third_party/WebKit/Source/core/frame/FrameView.cpp
@@ -2402,7 +2402,7 @@ void FrameView::updateWidgetPositionsIfNeeded()
updateWidgetPositions();
}
-void FrameView::updateAllLifecyclePhases(const LayoutRect* interestRect)
+void FrameView::updateAllLifecyclePhases(const IntRect* interestRect)
{
frame().localFrameRoot()->view()->updateLifecyclePhasesInternal(AllPhases, interestRect);
}
@@ -2418,7 +2418,7 @@ void FrameView::updateLifecycleToLayoutClean()
frame().localFrameRoot()->view()->updateLifecyclePhasesInternal(OnlyUpToLayoutClean, nullptr);
}
-void FrameView::updateLifecyclePhasesInternal(LifeCycleUpdateOption phases, const LayoutRect* interestRect)
+void FrameView::updateLifecyclePhasesInternal(LifeCycleUpdateOption phases, const IntRect* interestRect)
{
// This must be called from the root frame, since it recurses down, not up.
// Otherwise the lifecycles of the frames might be out of sync.
@@ -2484,7 +2484,7 @@ void FrameView::updatePaintProperties()
forAllFrameViews([](FrameView& frameView) { frameView.lifecycle().advanceTo(DocumentLifecycle::UpdatePaintPropertiesClean); });
}
-void FrameView::synchronizedPaint(const LayoutRect* interestRect)
+void FrameView::synchronizedPaint(const IntRect* interestRect)
{
ASSERT(RuntimeEnabledFeatures::slimmingPaintSynchronizedPaintingEnabled());
ASSERT(frame() == page()->mainFrame() || (!frame().tree().parent()->isLocalFrame()));
@@ -2506,16 +2506,13 @@ void FrameView::synchronizedPaint(const LayoutRect* interestRect)
});
}
-void FrameView::synchronizedPaintRecursively(GraphicsLayer* graphicsLayer, const LayoutRect* interestRect)
+void FrameView::synchronizedPaintRecursively(GraphicsLayer* graphicsLayer, const IntRect* interestRect)
{
ASSERT(graphicsLayer->paintController());
GraphicsContext context(*graphicsLayer->paintController());
// TODO(chrishtr): fix unit tests to not inject one-off interest rects.
- if (interestRect)
- graphicsLayer->paint(context, roundedIntRect(*interestRect));
- else
- graphicsLayer->paintIfNeeded(context);
+ graphicsLayer->paint(context, interestRect);
if (!RuntimeEnabledFeatures::slimmingPaintV2Enabled())
graphicsLayer->paintController()->commitNewDisplayItems();

Powered by Google App Engine
This is Rietveld 408576698