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

Unified Diff: third_party/WebKit/Source/platform/graphics/GraphicsLayer.cpp

Issue 2144303002: Made layout viewport scroll updates from compositor work like ordinary layers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@rootScrollerOnCompositor
Patch Set: Initialize ScrollUpdateInfo's layer id in constructor Created 4 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
Index: third_party/WebKit/Source/platform/graphics/GraphicsLayer.cpp
diff --git a/third_party/WebKit/Source/platform/graphics/GraphicsLayer.cpp b/third_party/WebKit/Source/platform/graphics/GraphicsLayer.cpp
index 45e1736605ae60aca446e877b3929e419dd73f8b..598edd9585a3494897d71226a3ee55f05652fcc6 100644
--- a/third_party/WebKit/Source/platform/graphics/GraphicsLayer.cpp
+++ b/third_party/WebKit/Source/platform/graphics/GraphicsLayer.cpp
@@ -1223,16 +1223,17 @@ void GraphicsLayer::removeLinkHighlight(LinkHighlight* linkHighlight)
updateChildList();
}
-void GraphicsLayer::setScrollableArea(ScrollableArea* scrollableArea, bool isViewport)
+void GraphicsLayer::setScrollableArea(ScrollableArea* scrollableArea, bool isVisualViewport)
{
if (m_scrollableArea == scrollableArea)
return;
m_scrollableArea = scrollableArea;
- // Viewport scrolling may involve pinch zoom and gets routed through
- // WebViewImpl explicitly rather than via GraphicsLayer::didScroll.
- if (isViewport)
+ // VisualViewport scrolling may involve pinch zoom and gets routed through
+ // WebViewImpl explicitly rather than via GraphicsLayer::didScroll since it
+ // needs to be set in tandem with the page scale delta.
+ if (isVisualViewport)
m_layer->layer()->setScrollClient(0);
else
m_layer->layer()->setScrollClient(this);
« no previous file with comments | « third_party/WebKit/Source/platform/graphics/GraphicsLayer.h ('k') | third_party/WebKit/Source/web/WebViewImpl.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698