Bug 105406 - New fast/events/touch/touch-input-element-change-documents.html fails on JSC platforms
Summary: New fast/events/touch/touch-input-element-change-documents.html fails on JSC ...
Status: REOPENED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on: 105452
Blocks: 87008
  Show dependency treegraph
 
Reported: 2012-12-19 02:05 PST by János Badics
Modified: 2012-12-19 11:57 PST (History)
9 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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