Bug 203218

Summary: REGRESSION (~r251067): http/tests/workers/service/registration-clear-redundant-worker.html is a flaky timeout
Product: WebKit Reporter: Russell Epstein <repstein>
Component: Tools / TestsAssignee: Nobody <webkit-unassigned>
Severity: Normal CC: webkit-bot-watchers-bugzilla, webkit-bug-importer, youennf
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Description Flags
Test List none

Description Russell Epstein 2019-10-21 15:53:41 PDT
The following layout test is a flaky timeout after the changes landed in r251067:


Test History:



--- /Volumes/Data/slave/ios-simulator-13-release-tests-wk2/build/layout-test-results/http/tests/workers/service/registration-clear-redundant-worker-expected.txt
+++ /Volumes/Data/slave/ios-simulator-13-release-tests-wk2/build/layout-test-results/http/tests/workers/service/registration-clear-redundant-worker-actual.txt
@@ -1,6 +1,5 @@
-PASS: Unregistration was successful
-PASS: Worker became redundant after unregistering
-PASS: registration.installing is null
-PASS: registration.waiting is null
-PASS: registration.active is null
+#PID UNRESPONSIVE - WebKitTestRunnerApp (pid 68680)
+FAIL: Timed out waiting for notifyDone to be called
Comment 1 Radar WebKit Bug Importer 2019-10-21 15:54:12 PDT
Comment 2 Russell Epstein 2019-10-21 16:34:06 PDT
Created attachment 381482 [details]
Test List

Test list for local reproduction.
Comment 3 Russell Epstein 2019-10-21 16:35:41 PDT
Steps to Reproduce:

Test appears timeout consistently when running the following command:

run-webkit-tests --ios-simulator --no-retry -g --no-sample-on-timeout --test-list test-list.txt --child-process 1

See test-list.txt attachment.
Comment 4 Russell Epstein 2019-10-21 16:38:33 PDT
(In reply to Russell Epstein from comment #0)
> Test History:
> https://results.webkit.org/?suite=layout-tests&suite=layout-
> tests&test=http%2Ftests%2Fsecurity%2FmixedContent%2Finsecure-image-in-iframe.
> html&test=http%2Ftests%2Fworkers%2Fservice%2Fregistration-clear-redundant-
> worker.html

Accidentally included extra test in history link.
Updated Link:

Comment 5 Russell Epstein 2019-10-21 16:52:29 PDT
Marked test as a flaky timeout in r251397
Comment 6 youenn fablet 2019-10-25 07:29:43 PDT
Thanks for the repro steps, that is really helpful!
I think I found the issue and will it as bug 203410.
Comment 7 youenn fablet 2019-10-25 07:30:02 PDT

*** This bug has been marked as a duplicate of bug 203410 ***