Kill worker process by way of a sync IPC message before it cleans up.

content/worker/websharedworker_stub.cc

Index: content/worker/websharedworker_stub.cc
diff --git a/content/worker/websharedworker_stub.cc b/content/worker/websharedworker_stub.cc
index 745b63c1956abf61d13cd564e4ef1667c38fa23b..79753eceed32dd2128ca76a581c832abb6522819 100644
--- a/content/worker/websharedworker_stub.cc
+++ b/content/worker/websharedworker_stub.cc
@@ -45,6 +45,14 @@ WebSharedWorkerStub::~WebSharedWorkerStub() {
   impl_->clientDestroyed();
   WorkerThread* worker_thread = WorkerThread::current();
   DCHECK(worker_thread);
+
+  // Send an IPC message to kill the worker process.
+  bool result = false;
+  worker_thread->Send(new WorkerProcessHostMsg_ForceKillWorker(

[michaeln, 2013/09/12 00:22:11]

I think WorkerThread::Shutdown() would be a better place to do this.

With this call here, the first shared worker to exit will cause the entire
process to die, including any other shared workers in that process.

Despite it's name, class WorkerThread is actually a one per process sort of
class.

[Mustafa Acer, 2013/09/12 00:29:14]

I started with that, but the route_id being removed from worker thread caused
the IPC to be lost. I'll take another look.

Also, from reading the code there seems to be a one to one mapping between
WorkerThread and WebSharedWorkerStub. Is that correct (probably not)?

+      route_id_,
+      base::GetCurrentProcessHandle(),
+      &result));
+
   worker_thread->RemoveWorkerStub(this);
   worker_thread->RemoveRoute(route_id_);
   ChildProcess::current()->ReleaseProcess();