Bug 160024 - [GTK] Layout test platform/gtk/scrollbars/main-frame-scrollbar-horizontal-wheel-scroll.html fails since r198269
Summary: [GTK] Layout test platform/gtk/scrollbars/main-frame-scrollbar-horizontal-whe...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on: 175674
Blocks:
  Show dependency treegraph
 
Reported: 2016-07-21 08:10 PDT by Carlos Alberto Lopez Perez
Modified: 2017-08-18 08:44 PDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Alberto Lopez Perez 2016-07-21 08:10:59 PDT
Since https://trac.webkit.org/changeset/198269 the layout test platform/gtk/scrollbars/main-frame-scrollbar-horizontal-wheel-scroll.html fails on platform GTK.

This is the diff image:

https://build.webkit.org/results/GTK%20Linux%2064-bit%20Release%20%28Tests%29/r203502%20%2817200%29/platform/gtk/scrollbars/main-frame-scrollbar-horizontal-wheel-scroll-diffs.html

As you can see, the expected results is that the window is scrolled to the right, but the actual result is not scrolling the window to the right.

Any of this two patches make the test pass:

--- a/LayoutTests/platform/gtk/scrollbars/main-frame-scrollbar-horizontal-wheel-scroll-expected.html
+++ b/LayoutTests/platform/gtk/scrollbars/main-frame-scrollbar-horizontal-wheel-scroll-expected.html
@@ -3,7 +3,7 @@
 <p>This is a test for bug <a href="https://bugs.webkit.org/show_bug.cgi?id=122859">122859</a>.
 Scrolling with the mouse wheel over the horizontal main scrollbar should scroll the view horizontally</p>
 <script>
-  document.scrollingElement.scrollLeft = 40;
+  document.scrollingElement.scrollLeft = 0;
 </script>
 </body>
 </html>


--- a/Source/WebCore/platform/ScrollAnimationSmooth.cpp
+++ b/Source/WebCore/platform/ScrollAnimationSmooth.cpp
@@ -254,7 +254,7 @@ static inline void getAnimationParametersForGranularity(ScrollGranularity granul
         maximumCoastTime = 1;
         break;
     case ScrollByPixel:
-        animationTime = 11 * tickTime;
+        animationTime = 1 * tickTime;
         repeatMinimumSustainTime = 2 * tickTime;
         attackTime = 3 * tickTime;
         releaseTime = 3 * tickTime;



However, I don't think any of them is right. 

Basically the first patch is disabling the horizontal scrolling on the expected result, and the second one is partially reverting r198269


So, I will mark this test as failing on the test expectations and leave here this bug.
Comment 1 Carlos Alberto Lopez Perez 2016-07-21 10:15:12 PDT
(In reply to comment #0)
> So, I will mark this test as failing on the test expectations and leave here
> this bug.

Marked here: https://trac.webkit.org/changeset/203509/trunk/LayoutTests/platform/gtk/TestExpectations