Bug 230382 – Implement SharedWorker

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Alex Christensen]

Implement SharedWorker

[Alex Christensen]

Created attachment 438425 [details]
Patch

[Geoffrey Garen]

Thread 15 Crashed:: WebCore: Worker
0   libsystem_kernel.dylib        	0x00007fff692bb33a __pthread_kill + 10
1   libsystem_pthread.dylib       	0x00007fff6937be60 pthread_kill + 430
2   libsystem_c.dylib             	0x00007fff69242808 abort + 120
3   libc++abi.dylib               	0x00007fff664ab458 abort_message + 231
4   libc++abi.dylib               	0x00007fff664aae92 __cxa_pure_virtual + 18
5   com.apple.WebCore             	0x0000000149d71ce9 WebCore::WorkerOrWorkletThread::workerOrWorkletThread() + 57
6   com.apple.JavaScriptCore      	0x000000014d624bbc WTF::Thread::entryPoint(WTF::Thread::NewThreadContext*) + 124
7   com.apple.JavaScriptCore      	0x000000014d627449 WTF::wtfThreadEntryPoint(void*) + 9
8   libsystem_pthread.dylib       	0x00007fff6937c109 _pthread_start + 148
9   libsystem_pthread.dylib       	0x00007fff69377b8b thread_start + 15

[Geoffrey Garen]

Comment on attachment 438425 [details]
Patch

EWS is angry

[Alex Christensen]

Created attachment 438521 [details]
Patch

[Alex Christensen]

Created attachment 438538 [details]
Patch

[Alex Christensen]

http://wpt.live/encoding/encodeInto.any.sharedworker.html asserts with this change, and I'm not sure why.

ASSERTION FAILED: !vm().currentThreadIsHoldingAPILock()

and doing this doesn't fix it

+  m_vm->apiLock().lock();
   m_vm->notifyNeedTermination();
+  m_vm->apiLock().unlock();

[Radar WebKit Bug Importer]

<rdar://problem/83474485>

[Chris Dumez]

(In reply to Alex Christensen from comment #6)
> http://wpt.live/encoding/encodeInto.any.sharedworker.html asserts with this
> change, and I'm not sure why.
> 
> ASSERTION FAILED: !vm().currentThreadIsHoldingAPILock()
> 
> and doing this doesn't fix it
> 
> +  m_vm->apiLock().lock();
>    m_vm->notifyNeedTermination();
> +  m_vm->apiLock().unlock();

It's the opposite, we should not be holding the lock when calling notifyNeedTermination(). I think this means that the worker thread gets marked for termination while in the middle on running script.

I don't have access to the full backtrace but I bet that you'll see that it's running script or doing some other operation that holds the JS lock.

[zhenya]

hi Alex, trust this finds you well. Is there any update on this ticket please?

[Daniel St-Germain]

Also interested in any updates. I'm currently working on a feature that requires sharedworkers.

[Chris Dumez]

(In reply to Daniel St-Germain from comment #10)
> Also interested in any updates. I'm currently working on a feature that
> requires sharedworkers.

I am actively working on this (via dependency bugs).

[Brent Fulgham]

This is now tracked as:
<rdar://64348204>

[Brent Fulgham]

*** Bug 149850 has been marked as a duplicate of this bug. ***

[Tobias Uhlig]

It is actually exciting to see that this ticket is flagged as resolved now :)

In which Safari Tech Preview is the SW support going to land?

In case you need a really advanced real world test-case:
https://neomjs.github.io/pages/node_modules/neo.mjs/apps/sharedcovid/index.html#mainview=table

Be aware though that the app will fall back to using non shared workers if needed. In case you can move components like the chart, helix or gallery into new browser windows, it is a success.

Demo video (Chrome):
https://youtu.be/aEA5333WiWY (starts at 2:40).

Thanks and best regards,
Tobias

[Brent Fulgham]

(In reply to Tobias Uhlig from comment #14)
> It is actually exciting to see that this ticket is flagged as resolved now :)
> 
> In which Safari Tech Preview is the SW support going to land?
> 
> In case you need a really advanced real world test-case:
> https://neomjs.github.io/pages/node_modules/neo.mjs/apps/sharedcovid/index.
> html#mainview=table
> 
> Be aware though that the app will fall back to using non shared workers if
> needed. In case you can move components like the chart, helix or gallery
> into new browser windows, it is a success.
> 
> Demo video (Chrome):
> https://youtu.be/aEA5333WiWY (starts at 2:40).
> 
> Thanks and best regards,
> Tobias

It usually takes a few weeks to get into an STP build. We will update this bug once the relevant STP build is available for download,