Fix main thread top controls scrolling to mirror CC.

third_party/WebKit/Source/web/ResizeViewportAnchor.cpp

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "web/ResizeViewportAnchor.h"
 
 #include "core/frame/FrameView.h"
 #include "core/frame/VisualViewport.h"
 #include "platform/geometry/DoubleRect.h"
 #include "platform/geometry/DoubleSize.h"
 #include "platform/geometry/FloatSize.h"
 
 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