Title: [277539] trunk/Source/WebCore
Revision
277539
Author
[email protected]
Date
2021-05-15 07:26:42 -0700 (Sat, 15 May 2021)

Log Message

[GTK] REGRESSION: Kinetic scrolling on touchpad doesn't work with async scrolling off
https://bugs.webkit.org/show_bug.cgi?id=224182

Patch by Alexander Mikhaylenko <[email protected]> on 2021-05-15
Reviewed by Michael Catanzaro.

Don't clear the scroll history every time we scroll, that defeats the whole point of having
the scroll history.

* platform/generic/ScrollAnimatorGeneric.cpp:
(WebCore::ScrollAnimatorGeneric::scrollToPositionWithoutAnimation):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (277538 => 277539)


--- trunk/Source/WebCore/ChangeLog	2021-05-15 12:47:21 UTC (rev 277538)
+++ trunk/Source/WebCore/ChangeLog	2021-05-15 14:26:42 UTC (rev 277539)
@@ -1,3 +1,16 @@
+2021-05-15  Alexander Mikhaylenko  <[email protected]>
+
+        [GTK] REGRESSION: Kinetic scrolling on touchpad doesn't work with async scrolling off
+        https://bugs.webkit.org/show_bug.cgi?id=224182
+
+        Reviewed by Michael Catanzaro.
+
+        Don't clear the scroll history every time we scroll, that defeats the whole point of having
+        the scroll history.
+
+        * platform/generic/ScrollAnimatorGeneric.cpp:
+        (WebCore::ScrollAnimatorGeneric::scrollToPositionWithoutAnimation):
+
 2021-05-15  Antti Koivisto  <[email protected]>
 
         Don't allow :visited link style in subtrees that use mix-blend-mode

Modified: trunk/Source/WebCore/platform/generic/ScrollAnimatorGeneric.cpp (277538 => 277539)


--- trunk/Source/WebCore/platform/generic/ScrollAnimatorGeneric.cpp	2021-05-15 12:47:21 UTC (rev 277538)
+++ trunk/Source/WebCore/platform/generic/ScrollAnimatorGeneric.cpp	2021-05-15 14:26:42 UTC (rev 277539)
@@ -65,7 +65,6 @@
 bool ScrollAnimatorGeneric::scrollToPositionWithoutAnimation(const FloatPoint& position, ScrollClamping clamping)
 {
     m_kineticAnimation->stop();
-    m_kineticAnimation->clearScrollHistory();
     return ScrollAnimator::scrollToPositionWithoutAnimation(position, clamping);
 }
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to