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

content/worker/websharedworker_stub.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/worker/websharedworker_stub.h"
 
 #include "base/compiler_specific.h"
 #include "content/child/child_process.h"
 #include "content/child/child_thread.h"
 #include "content/child/fileapi/file_system_dispatcher.h"
@@ skipping 27 matching lines @@
   // TODO(atwilson): Add support for NaCl when they support MessagePorts.
   impl_ = WebKit::WebSharedWorker::create(client());
   worker_devtools_agent_.reset(new SharedWorkerDevToolsAgent(route_id, impl_));
   client()->set_devtools_agent(worker_devtools_agent_.get());
 }
 
 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();
 }
 
 void WebSharedWorkerStub::Shutdown() {
   // The worker has exited - free ourselves and the client.
   delete this;
 }
 
@@ skipping 66 matching lines @@
 
 void WebSharedWorkerStub::OnTerminateWorkerContext() {
   impl_->terminateWorkerContext();
 
   // Call the client to make sure context exits.
   EnsureWorkerContextTerminates();
   started_ = false;
 }
 
 }  // namespace content