Closed Bug 1234042 Opened 10 years ago Closed 10 years ago

Intermittent TEST-UNEXPECTED-TIMEOUT | unregister-then-register-new-script.https.html | Registering a new script URL while an unregistered registration is in use - Test timed out

Categories

(Core :: DOM: Service Workers, defect)

Product:
Core
Component:
DOM: Service Workers
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: aryx, Assigned: bkelly)

References

Details

(Keywords: intermittent-failure)

https://treeherder.mozilla.org/logviewer.html#?job_id=18818253&repo=mozilla-inbound 16:01:44 INFO - PROCESS | 2956 | ###!!! [Parent][OnMaybeDequeueOne] Error: Channel closing: too late to send/recv, messages will be lost 16:01:44 INFO - PROCESS | 2956 | 16:01:44 INFO - PROCESS | 2956 | [2956] WARNING: A control runnable was posted to a worker that is already shutting down!: file c:/builds/moz2_slave/m-in-w32-d-0000000000000000000/build/src/dom/workers/WorkerPrivate.cpp, line 2337 16:01:44 INFO - PROCESS | 2956 | [2956] WARNING: A control runnable was posted to a worker that is already shutting down!: file c:/builds/moz2_slave/m-in-w32-d-0000000000000000000/build/src/dom/workers/WorkerPrivate.cpp, line 2337 16:01:44 INFO - PROCESS | 2956 | [2956] WARNING: A control runnable was posted to a worker that is already shutting down!: file c:/builds/moz2_slave/m-in-w32-d-0000000000000000000/build/src/dom/workers/WorkerPrivate.cpp, line 2337 16:01:47 INFO - PROCESS | 2956 | --DOMWINDOW == 23 (1A528C00) [pid = 2956] [serial = 3130] [outer = 00000000] [url = about:blank] 16:01:47 INFO - PROCESS | 2956 | --DOMWINDOW == 22 (1E76E400) [pid = 2956] [serial = 3128] [outer = 00000000] [url = about:blank] 16:01:47 INFO - PROCESS | 2956 | --DOMWINDOW == 21 (1A706400) [pid = 2956] [serial = 3132] [outer = 00000000] [url = about:blank] 16:01:47 INFO - PROCESS | 2956 | --DOMWINDOW == 20 (0F86F800) [pid = 2956] [serial = 3093] [outer = 00000000] [url = about:blank] 16:01:59 INFO - PROCESS | 2956 | MARIONETTE LOG: INFO: Timeout fired 16:01:59 INFO - TEST-UNEXPECTED-TIMEOUT | /_mozilla/service-workers/service-worker/unregister-then-register-new-script.https.html | Registering a new script URL while an unregistered registration is in use - Test timed out 16:01:59 INFO - TEST-PASS | /_mozilla/service-workers/service-worker/unregister-then-register-new-script.https.html | Registering a new script URL that 404s does resurrect an unregistered registration 16:01:59 INFO - TEST-PASS | /_mozilla/service-workers/service-worker/unregister-then-register-new-script.https.html | Registering a new script URL that fails to install does resurrect an unregistered registration 16:01:59 INFO - TEST-UNEXPECTED-TIMEOUT | /_mozilla/service-workers/service-worker/unregister-then-register-new-script.https.html | expected OK 16:01:59 INFO - TEST-INFO took 30372ms
Assignee: nobody → bkelly
Blocks: 1246319
Status: NEW → ASSIGNED
I think this may be fixed by the patches in bug 1246319. In particular, the second call to SendUnregister() was possibly causing havoc here.
No longer blocks: 1246319
Depends on: 1246319
Hmm, these recent failures are hitting: Registering a new script URL that 404s does resurrect an unregistered registration - assert_unreached: unexpected rejection: frame.contentWindow.navigator.serviceWorker.controller is null Reached unreachable code
Depends on: 1256428
Apparently the debugging output is enough to avoid the race triggering this failure. I think the best way to resolve this is to implement the new unified job queue in the spec. It should help us remove race conditions from the lifecycle code.
Speculatively closing this since bug 1256428 landed.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.