Bug 182603 - REGRESSION(r228260): [GTK] Broke several web workers tests
Summary: REGRESSION(r228260): [GTK] Broke several web workers tests
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-02-08 07:02 PST by Miguel Gomez
Modified: 2018-02-09 16:15 PST (History)
4 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Miguel Gomez 2018-02-08 07:02:49 PST
r228260 caused several tests to start failing, crashing or timing out:

imported/w3c/web-platform-tests/workers/WorkerGlobalScope_ErrorEvent_colno.htm [ Failure ]
imported/w3c/web-platform-tests/workers/WorkerGlobalScope_ErrorEvent_filename.htm [ Failure ]
imported/w3c/web-platform-tests/workers/WorkerGlobalScope_ErrorEvent_lineno.htm [ Failure ]
imported/w3c/web-platform-tests/workers/WorkerGlobalScope_ErrorEvent_message.htm [ Failure ]
imported/w3c/web-platform-tests/workers/Worker_ErrorEvent_error.htm [ Failure ]
imported/w3c/web-platform-tests/workers/interfaces/WorkerGlobalScope/onerror/exception-in-onerror.html [ Failure ]
imported/w3c/web-platform-tests/workers/interfaces/WorkerGlobalScope/onerror/handled.html [ Failure ]
storage/indexeddb/transaction-complete-workers-private.html [ Crash ]
storage/indexeddb/transaction-complete-workers.html [ Crash ]
fast/workers/worker-script-error.html [ Timeout ]
Comment 1 Michael Catanzaro 2018-02-08 08:33:00 PST
Can you post text diffs for the tests with failure results, and backtraces for the tests that crash? Otherwise, it's not likely that anybody will look at this bug.

Also, it's helpful to include the name of the commit alongside the revision number. These regressions were caused by r228260 "Event improvements". The commit name should help Darin understand why he was CCed. :)
Comment 2 Miguel Gomez 2018-02-09 00:31:00 PST
(In reply to Michael Catanzaro from comment #1)
> Can you post text diffs for the tests with failure results, and backtraces
> for the tests that crash? Otherwise, it's not likely that anybody will look
> at this bug.

Incoming diffs!!

imported/w3c/web-platform-tests/workers/WorkerGlobalScope_ErrorEvent_colno.htm [ Failure ]

--- /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/imported/w3c/web-platform-tests/workers/WorkerGlobalScope_ErrorEvent_colno-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/imported/w3c/web-platform-tests/workers/WorkerGlobalScope_ErrorEvent_colno-actual.txt
@@ -1,4 +1,6 @@
 CONSOLE MESSAGE: line 3: Error: Error Message
 
-PASS  WorkerGlobalScope onerror event handler argument: col  
+Harness Error (TIMEOUT), message = null
 
+TIMEOUT  WorkerGlobalScope onerror event handler argument: col  Test timed out
+

imported/w3c/web-platform-tests/workers/WorkerGlobalScope_ErrorEvent_filename.htm [ Failure ]

--- /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/imported/w3c/web-platform-tests/workers/WorkerGlobalScope_ErrorEvent_filename-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/imported/w3c/web-platform-tests/workers/WorkerGlobalScope_ErrorEvent_filename-actual.txt
@@ -1,4 +1,6 @@
 CONSOLE MESSAGE: line 3: Error: Error Message
 
-PASS  WorkerGlobalScope onerror event handler argument: location  
+Harness Error (TIMEOUT), message = null
 
+TIMEOUT  WorkerGlobalScope onerror event handler argument: location  Test timed out
+

imported/w3c/web-platform-tests/workers/WorkerGlobalScope_ErrorEvent_lineno.htm [ Failure ]

--- /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/imported/w3c/web-platform-tests/workers/WorkerGlobalScope_ErrorEvent_lineno-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/imported/w3c/web-platform-tests/workers/WorkerGlobalScope_ErrorEvent_lineno-actual.txt
@@ -1,4 +1,6 @@
 CONSOLE MESSAGE: line 3: Error: Error Message
 
-PASS  WorkerGlobalScope onerror event handler argument: line  
+Harness Error (TIMEOUT), message = null
 
+TIMEOUT  WorkerGlobalScope onerror event handler argument: line  Test timed out
+

imported/w3c/web-platform-tests/workers/WorkerGlobalScope_ErrorEvent_message.htm [ Failure ]

--- /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/imported/w3c/web-platform-tests/workers/WorkerGlobalScope_ErrorEvent_message-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/imported/w3c/web-platform-tests/workers/WorkerGlobalScope_ErrorEvent_message-actual.txt
@@ -1,4 +1,6 @@
 CONSOLE MESSAGE: line 3: Error: Error Message
 
-PASS  WorkerGlobalScope onerror event handler argument: message  
+Harness Error (TIMEOUT), message = null
 
+TIMEOUT  WorkerGlobalScope onerror event handler argument: message  Test timed out
+

imported/w3c/web-platform-tests/workers/Worker_ErrorEvent_error.htm [ Failure ]

--- /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/imported/w3c/web-platform-tests/workers/Worker_ErrorEvent_error-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/imported/w3c/web-platform-tests/workers/Worker_ErrorEvent_error-actual.txt
@@ -1,6 +1,8 @@
 CONSOLE MESSAGE: line 9: hello
 
+Harness Error (TIMEOUT), message = null
+
 PASS Error handler outside the worker should not see the error value 
-PASS Error handlers inside a worker should see the error value 
+TIMEOUT Error handlers inside a worker should see the error value Test timed out
 PASS Untitled 
 

imported/w3c/web-platform-tests/workers/interfaces/WorkerGlobalScope/onerror/exception-in-onerror.html [ Failure ]

--- /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/imported/w3c/web-platform-tests/workers/interfaces/WorkerGlobalScope/onerror/exception-in-onerror-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/imported/w3c/web-platform-tests/workers/interfaces/WorkerGlobalScope/onerror/exception-in-onerror-actual.txt
@@ -1,4 +1,3 @@
-CONSOLE MESSAGE: line 3: ReferenceError: Can't find variable: y
 
 PASS onerror, "not handled" with an error in the onerror function 
 

imported/w3c/web-platform-tests/workers/interfaces/WorkerGlobalScope/onerror/handled.html [ Failure ]

--- /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/imported/w3c/web-platform-tests/workers/interfaces/WorkerGlobalScope/onerror/handled-expected.txt
+++ /home/slave/webkitgtk/gtk-linux-64-release-tests/build/layout-test-results/imported/w3c/web-platform-tests/workers/interfaces/WorkerGlobalScope/onerror/handled-actual.txt
@@ -1,3 +1,4 @@
+CONSOLE MESSAGE: line 7: ReferenceError: Can't find variable: y
 
-PASS onerror, "handled" 
+FAIL onerror, "handled" assert_unreached: Reached unreachable code
Comment 3 Chris Dumez 2018-02-09 15:10:30 PST
I believe I fixed this in trac.webkit.org/r228276. Do you still experience these issues in the GTK port? The issues when away on Mac / iOS ports.
Comment 4 Chris Dumez 2018-02-09 15:14:58 PST
Tentatively marking as resolved by trac.webkit.org/changeset/228276/webkit as I do not see these tests failing on the GTK bots (anymore). Please reopen if there is still an issue for the GTK port caused by r228260 and I will help.
Comment 5 Michael Catanzaro 2018-02-09 16:15:16 PST
Thanks Chris! I bet you solved it, indeed. I'll remove the expectations.