[GTK] Test fast/forms/select-empty-option-height.html fails
authorcarlosgc@webkit.org <carlosgc@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 8 Feb 2017 16:25:43 +0000 (16:25 +0000)
committercarlosgc@webkit.org <carlosgc@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 8 Feb 2017 16:25:43 +0000 (16:25 +0000)
https://bugs.webkit.org/show_bug.cgi?id=167985

Reviewed by Michael Catanzaro.

This seems to be a problem with the differences in scroll step or the size of the items in the list. The thing
is that 10 attempts with an scroll amount of 10 is not enough in the GTk+ port to scroll the list to the desired
point. Either increasing the scroll amount or the maximum attempts would fix the test. Increasing the scroll
amount would break the test in other ports so better use a higher number of maximum attempts.

* fast/forms/listbox-respects-padding-bottom.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@211872 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/fast/forms/listbox-respects-padding-bottom.html

index 44f48d6..2719ab5 100644 (file)
@@ -1,5 +1,19 @@
 2017-02-08  Carlos Garcia Campos  <cgarcia@igalia.com>
 
+        [GTK] Test fast/forms/select-empty-option-height.html fails
+        https://bugs.webkit.org/show_bug.cgi?id=167985
+
+        Reviewed by Michael Catanzaro.
+
+        This seems to be a problem with the differences in scroll step or the size of the items in the list. The thing
+        is that 10 attempts with an scroll amount of 10 is not enough in the GTk+ port to scroll the list to the desired
+        point. Either increasing the scroll amount or the maximum attempts would fix the test. Increasing the scroll
+        amount would break the test in other ports so better use a higher number of maximum attempts.
+
+        * fast/forms/listbox-respects-padding-bottom.html:
+
+2017-02-08  Carlos Garcia Campos  <cgarcia@igalia.com>
+
         Unreviewed GTK+ gardening. Skip animations/font-variations/ tests.
 
         They are marked as ImageOnlyFailure, but are actually failing now. I think it's better to simply skip all font
index 45072be..db549bf 100644 (file)
@@ -6,7 +6,7 @@
         window.jsTestIsAsync = true;
 
         var i = 0;
-        var maxAttempts = 10;
+        var maxAttempts = 20;
         function runTest()
         {
             var scrollAmount = 0;