Fix smooth scroll jump when switching scroll handling between MT and CC

third_party/WebKit/Source/platform/scroll/ScrollAnimatorCompositorCoordinator.cpp

 // Copyright 2014 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 "platform/scroll/ScrollAnimatorCompositorCoordinator.h"
 
 #include "platform/RuntimeEnabledFeatures.h"
 #include "platform/graphics/GraphicsLayer.h"
+#include "platform/scroll/MainThreadScrollingReason.h"
 #include "platform/scroll/ScrollableArea.h"
 #include "public/platform/Platform.h"
 #include "public/platform/WebCompositorAnimationPlayer.h"
 #include "public/platform/WebCompositorAnimationTimeline.h"
 #include "public/platform/WebCompositorSupport.h"
 
 namespace blink {
 
+WebLayer* toWebLayer(blink::GraphicsLayer* layer)
+{
+    return layer ? layer->platformLayer() : nullptr;
+}
+
 ScrollAnimatorCompositorCoordinator::ScrollAnimatorCompositorCoordinator()
     : m_compositorAnimationAttachedToLayerId(0)
     , m_runState(RunState::Idle)
     , m_compositorAnimationId(0)
     , m_compositorAnimationGroupId(0)
+    , m_clearMainThreadScrollingReasons(false)
 {
     if (RuntimeEnabledFeatures::compositorAnimationTimelinesEnabled()) {
         ASSERT(Platform::current()->compositorSupport());
         m_compositorPlayer = adoptPtr(Platform::current()->compositorSupport()->createAnimationPlayer());
         ASSERT(m_compositorPlayer);
         m_compositorPlayer->setAnimationDelegate(this);
     }
 }
 
 ScrollAnimatorCompositorCoordinator::~ScrollAnimatorCompositorCoordinator()
 {
     if (m_compositorPlayer) {
         m_compositorPlayer->setAnimationDelegate(nullptr);
         m_compositorPlayer.clear();
     }
 }
 
 void ScrollAnimatorCompositorCoordinator::resetAnimationState()
 {
     m_runState = RunState::Idle;
     m_compositorAnimationId = 0;
     m_compositorAnimationGroupId = 0;
+
+    // While there was a running animation, a temporary main thread scrolling
+    // reason may have been added for the animation to complete on the main
+    // thread.
+    clearMainThreadScrollingReasonsOnLayer();
+}
+
+void ScrollAnimatorCompositorCoordinator::clearMainThreadScrollingReasonsOnLayer()
+{
+    if (WebLayer* scrollLayer = toWebLayer(scrollableArea()->layerForScrolling())) {
+        if (m_clearMainThreadScrollingReasons) {
+            scrollLayer->clearMainThreadScrollingReasons();
+            m_clearMainThreadScrollingReasons = false;
+        } else {
+            scrollLayer->clearMainThreadScrollingReasons(
+                MainThreadScrollingReason::kContinuingMainThreadScroll);
+        }
+    }
+}
+
+void ScrollAnimatorCompositorCoordinator::clearMainThreadScrollingReasons(bool clear)
+{
+    m_clearMainThreadScrollingReasons = clear;
+    if (!clear)
+        return;
+    if (!hasRunningAnimation()) {
+        clearMainThreadScrollingReasonsOnLayer();
+    } else if (WebLayer* scrollLayer = toWebLayer(scrollableArea()->layerForScrolling())) {
+        // If there is a running animation, finish it on main thread before
+        // clearing main thread scrolling reasons. In the mean time, add
+        // a temporary main thread scrolling reason that ensures that the cc
+        // doesn't start another scroll animation that will compete with this
+        // animation (see crbug.com/581875).
+        scrollLayer->addMainThreadScrollingReasons(
+            MainThreadScrollingReason::kContinuingMainThreadScroll);
+    }
 }
 
 bool ScrollAnimatorCompositorCoordinator::hasAnimationThatRequiresService() const
 {
     switch (m_runState) {
     case RunState::Idle:
     case RunState::RunningOnCompositor:
         return false;
     case RunState::WaitingToSendToCompositor:
     case RunState::RunningOnMainThread:
@@ skipping 135 matching lines @@
     // animation.
     notifyCompositorAnimationFinished(group);
 }
 
 WebCompositorAnimationPlayer* ScrollAnimatorCompositorCoordinator::compositorPlayer() const
 {
     return m_compositorPlayer.get();
 }
 
 } // namespace blink