Correctly update scroll offset animations in response to scroll anchoring

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

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
@@ skipping 65 matching lines @@
     MOCK_METHOD0(scheduleAnimation, bool());
 
     bool userInputScrollable(ScrollbarOrientation) const override { return true; }
     bool shouldPlaceVerticalScrollbarOnLeft() const override { return false; }
     IntPoint scrollPosition() const override { return IntPoint(); }
     int visibleHeight() const override { return 768; }
     int visibleWidth() const override { return 1024; }
     bool scrollAnimatorEnabled() const override { return m_scrollAnimatorEnabled; }
     int pageStep(ScrollbarOrientation) const override { return 0; }
 
+    void setScrollAnimator(ScrollAnimator* scrollAnimator)
+    {
+        animator = scrollAnimator;
+    }
+
+    DoublePoint scrollPositionDouble() const override
+    {
+        if (animator)
+            return animator->currentPosition();
+        return ScrollableArea::scrollPositionDouble();
+    }
+
+    void setScrollPosition(const DoublePoint& position, ScrollType type,
+        ScrollBehavior behavior = ScrollBehaviorInstant)
+    {
+        if (animator)
+            animator->setCurrentPosition(toFloatPoint(position));
+        ScrollableArea::setScrollPosition(position, type, behavior);
+    }
+
     DEFINE_INLINE_VIRTUAL_TRACE()
     {
+        visitor->trace(animator);
         ScrollableArea::trace(visitor);
     }
 
 private:
     explicit MockScrollableArea(bool scrollAnimatorEnabled)
         : m_scrollAnimatorEnabled(scrollAnimatorEnabled) { }
 
     bool m_scrollAnimatorEnabled;
+    Member<ScrollAnimator> animator;
 };
 
 class TestScrollAnimator : public ScrollAnimator {
 public:
     TestScrollAnimator(ScrollableArea* scrollableArea, WTF::TimeFunction timingFunction)
         : ScrollAnimator(scrollableArea, timingFunction) {};
     ~TestScrollAnimator() override {};
 
     void setShouldSendToCompositor(bool send)
     {
@@ skipping 421 matching lines @@
     TestScrollAnimator* animator = new TestScrollAnimator(scrollableArea, getMockedTime);
 
     // From calls to adjust/takeoverImplOnlyScrollOffsetAnimation.
     EXPECT_CALL(*scrollableArea, registerForAnimation()).Times(3);
 
     // Verify that the adjustment update is cleared.
     EXPECT_EQ(animator->m_runState, ScrollAnimatorCompositorCoordinator::RunState::Idle);
     EXPECT_FALSE(animator->hasAnimationThatRequiresService());
     EXPECT_TRUE(animator->implOnlyAnimationAdjustmentForTesting().isZero());
 
-    animator->adjustImplOnlyScrollOffsetAnimation(FloatSize(100.f, 100.f));
-    animator->adjustImplOnlyScrollOffsetAnimation(FloatSize(10.f, -10.f));
+    animator->adjustImplOnlyScrollOffsetAnimation(IntSize(100, 100));
+    animator->adjustImplOnlyScrollOffsetAnimation(IntSize(10, -10));
 
     EXPECT_TRUE(animator->hasAnimationThatRequiresService());
-    EXPECT_EQ(FloatSize(110.f, 90.f), animator->implOnlyAnimationAdjustmentForTesting());
+    EXPECT_EQ(FloatSize(110, 90), animator->implOnlyAnimationAdjustmentForTesting());
 
     animator->updateCompositorAnimations();
 
     EXPECT_EQ(animator->m_runState, ScrollAnimatorCompositorCoordinator::RunState::Idle);
     EXPECT_FALSE(animator->hasAnimationThatRequiresService());
     EXPECT_TRUE(animator->implOnlyAnimationAdjustmentForTesting().isZero());
 
     // Verify that the takeover update is cleared.
     animator->takeOverImplOnlyScrollOffsetAnimation();
     EXPECT_TRUE(animator->hasAnimationThatRequiresService());
     animator->updateCompositorAnimations();
     EXPECT_FALSE(animator->hasAnimationThatRequiresService());
 
     // Forced GC in order to finalize objects depending on the mock object.
     ThreadHeap::collectAllGarbage();
 }
 
+TEST(ScrollAnimatorTest, MainThreadAnimationTargetAdjustment)
+{
+    MockScrollableArea* scrollableArea = MockScrollableArea::create(true);
+    ScrollAnimator* animator = new ScrollAnimator(scrollableArea, getMockedTime);
+    scrollableArea->setScrollAnimator(animator);
+
+    EXPECT_CALL(*scrollableArea, minimumScrollPosition()).Times(AtLeast(1))
+        .WillRepeatedly(Return(IntPoint(-100, -100)));
+    EXPECT_CALL(*scrollableArea, maximumScrollPosition()).Times(AtLeast(1))
+        .WillRepeatedly(Return(IntPoint(1000, 1000)));
+    // Twice from tickAnimation, once from reset, and once from
+    // adjustAnimationAndSetScrollPosition.
+    EXPECT_CALL(*scrollableArea, setScrollOffset(_, _)).Times(4);
+    // One from call to userScroll and one from updateCompositorAnimations.
+    EXPECT_CALL(*scrollableArea, registerForAnimation()).Times(2);
+    EXPECT_CALL(*scrollableArea, scheduleAnimation()).Times(AtLeast(1))
+        .WillRepeatedly(Return(true));
+
+    // Idle
+    EXPECT_FALSE(animator->hasAnimationThatRequiresService());
+    EXPECT_EQ(FloatPoint(), animator->currentPosition());
+
+    // WaitingToSendToCompositor
+    animator->userScroll(ScrollByLine, FloatSize(100, 100));
+
+    // RunningOnMainThread
+    gMockedTime += 0.05;
+    animator->updateCompositorAnimations();
+    animator->tickAnimation(getMockedTime());
+    FloatPoint pos = animator->currentPosition();
+    EXPECT_EQ(FloatPoint(100, 100), animator->desiredTargetPosition());
+    EXPECT_GT(pos.x(), 0);
+    EXPECT_GT(pos.y(), 0);
+
+    // Adjustment
+    IntSize adjustment = IntSize(10, -10);
+    animator->adjustAnimationAndSetScrollPosition(adjustment, AnchoringScroll);
+    EXPECT_EQ(pos + FloatSize(adjustment), animator->currentPosition());
+    EXPECT_EQ(FloatPoint(110, 90), animator->desiredTargetPosition());
+
+    // Animation finished
+    gMockedTime += 1.0;
+    animator->updateCompositorAnimations();
+    animator->tickAnimation(getMockedTime());
+    EXPECT_EQ(FloatPoint(110, 90), animator->currentPosition());
+    reset(*animator);
+
+    // Forced GC in order to finalize objects depending on the mock object.
+    ThreadHeap::collectAllGarbage();
+}
+
 } // namespace blink