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

Unified Diff: sky/engine/core/rendering/RenderView.cpp

Issue 646273006: Get rid of ScrollView. (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: merge to ToT Created 6 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: sky/engine/core/rendering/RenderView.cpp
diff --git a/sky/engine/core/rendering/RenderView.cpp b/sky/engine/core/rendering/RenderView.cpp
index 5c6d936d541d00a5508ba23f60f878603109b1ae..8f5a7dd127c4577ff3aae2fb25c20b94b3263a3a 100644
--- a/sky/engine/core/rendering/RenderView.cpp
+++ b/sky/engine/core/rendering/RenderView.cpp
@@ -249,7 +249,7 @@ void RenderView::computeSelfHitTestRects(Vector<LayoutRect>& rects, const Layout
// Record the entire size of the contents of the frame. Note that we don't just
// use the viewport size (containing block) here because we want to ensure this includes
// all children (so we can avoid walking them explicitly).
- rects.append(LayoutRect(LayoutPoint::zero(), frameView()->contentsSize()));
+ rects.append(LayoutRect(LayoutPoint::zero(), frameView()->size()));
}
void RenderView::paint(PaintInfo& paintInfo, const LayoutPoint& paintOffset)
@@ -378,10 +378,6 @@ void RenderView::mapRectToPaintInvalidationBacking(const RenderLayerModelObject*
if (viewportConstraint == IsFixedPosition && m_frameView) {
rect.move(m_frameView->scrollOffsetForFixedPosition());
- // If we have a pending scroll, invalidate the previous scroll position.
- if (!m_frameView->pendingScrollDelta().isZero()) {
- rect.move(-m_frameView->pendingScrollDelta());
- }
}
// Apply our transform if we have one (because of full page zooming).

Powered by Google App Engine
This is Rietveld 408576698