Worker and websocket close fix

[dylan-conway]

What does this PR do?

Worker:

• set m_wasTerminated to true after sending close event
• switch to ThreadSafeRefCounted fromRefCounted
• remove ref count check from hasPendingActivity in worker

These changes fix bugs causing pure virtual function called! errors when using workers

Also for on_close for websockets, check if message is null instead of only length > 0 before indexing message

How did you verify your code works?

manually and existing tests

[github-actions]

❌ @dylan-conway, your commit has failing tests :(

💻 1 failing tests Darwin x64 baseline

• test/js/web/streams/streams.test.js 1 failing

🪟💻 3 failing tests Windows x64 baseline

• test/cli/install/bunx.test.ts 1 failing
• test/integration/next-pages/test/next-build.test.ts 1 failing
• test/js/node/watch/fs.watchFile.test.ts 3 failing

🪟💻 5 failing tests Windows x64

• test/cli/install/bunx.test.ts 1 failing
• test/cli/install/migration/out-of-sync.test.ts 1 failing
• test/cli/install/registry/bun-install-registry.test.ts 1 failing
• test/integration/next-pages/test/dev-server.test.ts 1 failing
• test/js/node/watch/fs.watchFile.test.ts 3 failing

View logs