Bug 106318 - New fast/events/drag-and-drop-autoscroll.html fails
Summary: New fast/events/drag-and-drop-autoscroll.html fails
Status: REOPENED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 420+
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: Qt, QtTriaged
Depends on:
Blocks: 39725 87008
  Show dependency tree / graph
 
Reported: 2013-01-08 03:37 PST by Csaba Osztrogonác
Modified: 2013-04-11 11:41 PDT (History)
7 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Csaba Osztrogonác 2013-01-08 03:37:45 PST 
fast/events/drag-and-drop-autoscroll.html introduced in 
https://trac.webkit.org/changeset/139044 and fails on Qt and EFL:

--- /ramdisk/qt-linux-64-release/build/layout-test-results/fast/events/drag-and-drop-autoscroll-expected.txt 
+++ /ramdisk/qt-linux-64-release/build/layout-test-results/fast/events/drag-and-drop-autoscroll-actual.txt 
@@ -3,6 +3,6 @@
 On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
 
 
-PASS scrollable.scrollTop > 0
+FAIL No autoscroll
 For manual testing, drag and drop "Drop Me" to "Scrollable" area.
Comment 1 Csaba Osztrogonác 2013-01-08 04:40:10 PST 
I skipped it on Qt to paint the bots green - https://trac.webkit.org/changeset/139051

Please unskip it with the proper fix.
Comment 2 Hajime Morrita 2013-01-08 16:36:39 PST 
(In reply to comment #1)
> I skipped it on Qt to paint the bots green - https://trac.webkit.org/changeset/139051
> 
> Please unskip it with the proper fix.

Thanks for taking care of this.
Comment 3 Dominik Röttsches (drott) 2013-01-16 01:48:50 PST 
Rolled out in r139503.
Comment 4 Hajime Morrita 2013-01-16 02:17:56 PST 
(In reply to comment #3)
> Rolled out in r139503.
Ouch, I overlooked this. Thanks.
Comment 5 Tim Horton 2013-01-18 00:37:31 PST 
Do we need this bug anymore? The *test* was rolled out.
Comment 6 yosin 2013-01-18 00:39:58 PST 
(In reply to comment #5)
> Do we need this bug anymore? The *test* was rolled out.
No.

Although, the new patch is now reviewing. So, it will be land again.
Comment 7 Stephen Chenney 2013-04-08 15:48:01 PDT 
Oops, this is explicitly not Chromium.