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 / Tests | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED CONFIGURATION CHANGED | ||
| Severity: | Normal | CC: | ap, 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 | ||
János Badics
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.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
János Badics
(the original bug can be found at https://bugs.webkit.org/show_bug.cgi?id=105334)
Csaba Osztrogonác
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.
János Badics
The test has been skipped in r138134.
Csaba Osztrogonác
(In reply to comment #3)
> The test has been skipped in r138134.
only on Qt ...
Please unskip it with the proper fix.
Levi Weintraub
Fixed the test and unskipped in r138178.
Csaba Osztrogonác
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
Csaba Osztrogonác
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
Alexey Proskuryakov
Qt and EFL are no longer supported ports.