WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
Bug 25255
Support termination of nested workers.
https://bugs.webkit.org/show_bug.cgi?id=25255
Summary
Support termination of nested workers.
Jian Li
Reported
2009-04-16 17:43:13 PDT
Make sure workers and nested workers are terminated correctly.
Attachments
Proposed Patch
(7.91 KB, patch)
2009-04-16 17:52 PDT
,
Jian Li
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Jian Li
Comment 1
2009-04-16 17:52:10 PDT
Created
attachment 29567
[details]
Proposed Patch
Jian Li
Comment 2
2009-04-16 18:12:53 PDT
The following illustrates how we shut down nested worker and worker: 1) When a worker is terminated, WorkerThread::stop() is called. If there exists nested workers, we will try to stop nested workers first. This is done by calling m_workerContext->stopActiveDOMObjects(). 2) When the nested worker thread is tearing down and nested worker context is destroyed, WorkerMessagingProxy::workerContextDestroyed is called. From here, the parent worker context is notified. 3) When the parent worker context does not have nested worker threads running any more, it starts to shut down its own worker thread.
Ahmad Saleem
Comment 3
2022-08-10 16:58:06 PDT
I think this is still an issue because we do fail WPT tests:
https://wpt.fyi/results/workers?label=master&label=experimental&aligned&view=subtest&q=nested%20workers
Please correct me for my own learning, if I am wrong. Changing status to "New". In case, if this is different case and it is "RESOLVED WONTFIX" or "RESOLVED INVALID" or "RESOLVED CONFIGURATION CHANGED", please update accordingly. Thanks!
Chris Dumez
Comment 4
2022-08-10 17:29:17 PDT
AFAIK we do not support nested workers at all.
Chris Dumez
Comment 5
2022-09-17 19:39:49 PDT
https://commits.webkit.org/254597@main
Radar WebKit Bug Importer
Comment 6
2022-09-17 19:40:51 PDT
<
rdar://problem/100079574
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug