New test editing/selection/drag-list-item.html is failing

[Zoltan Arvai]

Newly added test editing/selection/drag-list-item.html in r145195 is failing on Qt WK2 and EFL WK2.
It times out on Qt WK1.

http://build.webkit.sed.hu/results/x86-64%20Linux%20Qt%20Release%20WebKit2%20%28Amazon%20EC2%29/r145214%20%2814663%29/results.html

--- /home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/layout-test-results/editing/selection/drag-list-item-expected.txt
+++ /home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/layout-test-results/editing/selection/drag-list-item-actual.txt
@@ -26,16 +26,13 @@
 |   "one"
 | <li>
 |   id="two"
-|   <br>
+|   "two"
 | <li>
 |   id="three"
 |   "three"
 | <li>
 |   id="four"
 |   "four"
-| <li>
-|   id="two"
-|   "two"
 | "
 "
 
@@ -59,13 +56,10 @@
 |   "one"
 | <li>
 |   id="two"
-|   <br>
-| <li>
-|   id="four"
-|   "four"
-| <li>
-|   id="two"
 |   "two"
 | <li>
 |   id="three"
 |   "three"
+| <li>
+|   id="four"
+|   "four"

[Sergio Villar Senin]

(In reply to comment #0)
> Newly added test editing/selection/drag-list-item.html in r145195 is failing on Qt WK2 and EFL WK2.
> It times out on Qt WK1.
> 
> http://build.webkit.sed.hu/results/x86-64%20Linux%20Qt%20Release%20WebKit2%20%28Amazon%20EC2%29/r145214%20%2814663%29/results.html
> 
> --- /home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/layout-test-results/editing/selection/drag-list-item-expected.txt
> +++ /home/webkitbuildbot/slaves/release64bitWebKit2_EC2/buildslave/qt-linux-64-release-webkit2/build/layout-test-results/editing/selection/drag-list-item-actual.txt
> @@ -26,16 +26,13 @@
>  |   "one"
>  | <li>
>  |   id="two"
> -|   <br>
> +|   "two"
>  | <li>
>  |   id="three"
>  |   "three"
>  | <li>
>  |   id="four"
>  |   "four"
> -| <li>
> -|   id="two"
> -|   "two"
>  | "
>  "
> 
> @@ -59,13 +56,10 @@
>  |   "one"
>  | <li>
>  |   id="two"
> -|   <br>
> -| <li>
> -|   id="four"
> -|   "four"
> -| <li>
> -|   id="two"
>  |   "two"
>  | <li>
>  |   id="three"
>  |   "three"
> +| <li>
> +|   id="four"
> +|   "four"

So it looks like the drag&drop is not taking place, maybe it needs a higher leapForward(). Could you check if it works using, say a value of 300?

[Sergio Villar Senin]

(In reply to comment #1)
> So it looks like the drag&drop is not taking place, maybe it needs a higher leapForward(). Could you check if it works using, say a value of 300?

It's not a matter of the leapForward but I found this in qt's TestExpectations file:

# Drag n drop support on DRT cannot work at the moment because Qt's drag'n'drop
# relies on an X event getting to the main window. In the absence of being able
# to receive this event, running windowless, DRT cannot tell what is being dropped and where.

So yeah I guess the right decission is to skip it until QtWebKit gets drag&drop support for testing.