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

Unified Diff: Source/core/rendering/compositing/CompositedLayerMapping.cpp

Issue 667913003: Reparented overflow controls host layer must account for scrolling. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Adding layout test Created 6 years, 1 month 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: Source/core/rendering/compositing/CompositedLayerMapping.cpp
diff --git a/Source/core/rendering/compositing/CompositedLayerMapping.cpp b/Source/core/rendering/compositing/CompositedLayerMapping.cpp
index 3cf7bbc68826b66f78d5bed3a538e2beadbbc67b..40aa11be064e907c9f420e1377ae8c27e3a3cba1 100644
--- a/Source/core/rendering/compositing/CompositedLayerMapping.cpp
+++ b/Source/core/rendering/compositing/CompositedLayerMapping.cpp
@@ -801,6 +801,8 @@ void CompositedLayerMapping::updateOverflowControlsHostLayerGeometry(const Rende
m_owningLayer.renderer()->mapLocalToContainer(compositingStackingContext->renderer(), transformState, ApplyContainerFlip);
transformState.flatten();
LayoutPoint offsetFromStackingContainer = LayoutPoint(transformState.lastPlanarPoint());
+ if (RenderLayerScrollableArea* scrollableArea = compositingStackingContext->scrollableArea())
+ offsetFromStackingContainer.move(LayoutSize(scrollableArea->adjustedScrollOffset()));
m_overflowControlsHostLayer->setPosition(FloatPoint(offsetFromStackingContainer));
}
} else {

Powered by Google App Engine
This is Rietveld 408576698