Fix main thread top controls scrolling to mirror CC.

third_party/WebKit/Source/web/ResizeViewportAnchor.cpp

Index: third_party/WebKit/Source/web/ResizeViewportAnchor.cpp
diff --git a/third_party/WebKit/Source/web/ResizeViewportAnchor.cpp b/third_party/WebKit/Source/web/ResizeViewportAnchor.cpp
index 8c79f083d325094e0a910dd469f64936fe9b733a..cac7353a972c926c4957e479e2b240f58fbf4f32 100644
--- a/third_party/WebKit/Source/web/ResizeViewportAnchor.cpp
+++ b/third_party/WebKit/Source/web/ResizeViewportAnchor.cpp
@@ -14,13 +14,46 @@ namespace blink {
 
 ResizeViewportAnchor::ResizeViewportAnchor(FrameView& rootFrameView, VisualViewport& visualViewport)
     : ViewportAnchor(rootFrameView, visualViewport)
-    , m_visualViewportInDocument(rootFrameView.getScrollableArea()->visibleContentRectDouble().location())
+    , m_visualViewportInDocument(rootFrameView.getScrollableArea()->scrollPositionDouble())
 {
 }
 
 ResizeViewportAnchor::~ResizeViewportAnchor()
 {
-    m_rootFrameView->getScrollableArea()->setScrollPosition(m_visualViewportInDocument, ProgrammaticScroll);
+    // TODO(bokan): Don't use RootFrameViewport::setScrollPosition since it
+    // assumes we can just set a sub-pixel precision offset on the FrameView.
+    // While we "can" do this, the offset that will be shipped to CC will be the
+    // truncated number and this class is used to handle TopControl movement
+    // which needs the two threads to match exactly pixel-for-pixel. We can
+    // replace this with RFV::setScrollPosition once Blink is sub-pixel scroll
+    // offset aware. crbug.com/414283.
+
+    ScrollableArea* rootViewport = m_rootFrameView->getScrollableArea();
+    ScrollableArea* layoutViewport =
+        m_rootFrameView->layoutViewportScrollableArea();
+
+    DoubleSize delta = m_visualViewportInDocument
+        - rootViewport->scrollPositionDouble();
+
+    m_visualViewport->move(toFloatSize(delta));
+
+    // Since the main thread FrameView has integer scroll offsets, scroll it to
+    // the next pixel and then we'll scroll the visual viewport again to
+    // compensate for the sub-pixel offset.
+    delta = m_visualViewportInDocument
+        - rootViewport->scrollPositionDouble();
+
+    IntSize layoutDelta = delta.height() < 0
+        ? flooredIntSize(delta)
+        : expandedIntSize(delta);
+
+    layoutViewport->setScrollPosition(
+        layoutViewport->scrollPosition() + layoutDelta,
+        ProgrammaticScroll);
+
+    delta = m_visualViewportInDocument
+        - rootViewport->scrollPositionDouble();
+    m_visualViewport->move(toFloatSize(delta));
 }
 
 } // namespace blink