Bug 105406

Summary: New fast/events/touch/touch-input-element-change-documents.html fails on JSC platforms
Product: WebKit Reporter: János Badics <jbadics>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Status: REOPENED ---    
Severity: Normal CC: d-r, hnandor, jamesr, kadam, kkristof, laszlo.gombos, leviw, ossy, zan
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 105452    
Bug Blocks: 87008    

Description János Badics 2012-12-19 02:05:56 PST
This test has been failing since it was introduced in http://trac.webkit.org/changeset/138095.
It has been corrected in https://trac.webkit.org/changeset/138115, but it passed only on chromium after the change.
It still fails on the other platforms where fast/events/touch are not skipped.
Qt fail diff:
http://build.webkit.org/results/Qt%20Linux%20Release/r138130%20(55683)/fast/events/touch/touch-input-element-change-documents-diff.txt
EFL fail diff:
http://build.webkit.org/results/EFL%20Linux%2064-bit%20Release/r138125%20(8530)/fast/events/touch/touch-input-element-change-documents-diff.txt

I will skip this test until it is fixed.
Comment 1 János Badics 2012-12-19 02:09:55 PST
(the original bug can be found at https://bugs.webkit.org/show_bug.cgi?id=105334)
Comment 2 Csaba Osztrogonác 2012-12-19 02:19:09 PST
I don't think if it is Qt and/or EFL specific. It fails on Qt and EFL, but doesn't on Mac and GTK, because all fast/events/touch are skipped on them.
Comment 3 János Badics 2012-12-19 02:33:53 PST
The test has been skipped in r138134.
Comment 4 Csaba Osztrogonác 2012-12-19 02:42:32 PST
(In reply to comment #3)
> The test has been skipped in r138134.
only on Qt ...

Please unskip it with the proper fix.
Comment 5 Levi Weintraub 2012-12-19 09:51:09 PST
Fixed the test and unskipped in r138178.
Comment 6 Csaba Osztrogonác 2012-12-19 11:51:03 PST
Reopen, because fast/events/touch/touch-input-element-change-documents.html still fails on Qt:

--- /ramdisk/qt-linux-64-release/build/layout-test-results/fast/events/touch/touch-input-element-change-documents-expected.txt 
+++ /ramdisk/qt-linux-64-release/build/layout-test-results/fast/events/touch/touch-input-element-change-documents-actual.txt 
@@ -5,8 +5,8 @@
 
 PASS window.internals.touchEventHandlerCount(document) is 0
 PASS window.internals.touchEventHandlerCount(document) is 0
-PASS window.internals.touchEventHandlerCount(document) is 2
-PASS window.internals.touchEventHandlerCount(document) is 3
-PASS window.internals.touchEventHandlerCount(document) is 2
+FAIL window.internals.touchEventHandlerCount(document) should be 2. Was 3.
+FAIL window.internals.touchEventHandlerCount(document) should be 3. Was 4.
+FAIL window.internals.touchEventHandlerCount(document) should be 2. Was 3.
 PASS window.internals.touchEventHandlerCount(document) is 0
Comment 7 Csaba Osztrogonác 2012-12-19 11:52:02 PST
and still fails on EFL:

--- /mnt/buildbot/efl-linux-slave-1/efl-linux-64-release/build/layout-test-results/fast/events/touch/touch-input-element-change-documents-expected.txt
+++ /mnt/buildbot/efl-linux-slave-1/efl-linux-64-release/build/layout-test-results/fast/events/touch/touch-input-element-change-documents-actual.txt
@@ -5,8 +5,8 @@
 
 PASS window.internals.touchEventHandlerCount(document) is 0
 PASS window.internals.touchEventHandlerCount(document) is 0
-PASS window.internals.touchEventHandlerCount(document) is 2
-PASS window.internals.touchEventHandlerCount(document) is 3
-PASS window.internals.touchEventHandlerCount(document) is 2
+FAIL window.internals.touchEventHandlerCount(document) should be 2. Was 0.
+FAIL window.internals.touchEventHandlerCount(document) should be 3. Was 1.
+FAIL window.internals.touchEventHandlerCount(document) should be 2. Was 0.
 PASS window.internals.touchEventHandlerCount(document) is 0