Bug 111853

Summary: New test editing/selection/drag-list-item.html is failing
Product: WebKit Reporter: Zoltan Arvai <zarvai>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: NEW ---    
Severity: Normal CC: kadam, rniwa, shezbaig.wk, svillar
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 87008, 111556    

Description Zoltan Arvai 2013-03-08 07:04:04 PST
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"
Comment 1 Sergio Villar Senin 2013-03-08 08:01:37 PST
(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?
Comment 2 Sergio Villar Senin 2013-03-08 09:04:48 PST
(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.