ServiceWorker: Implement StartWorker by using mojo

content/renderer/render_thread_impl.cc

Index: content/renderer/render_thread_impl.cc
diff --git a/content/renderer/render_thread_impl.cc b/content/renderer/render_thread_impl.cc
index 796e9b92a6c76ec2feb6c6595998f6a2006bee5c..ff9183752fc88c665e0fc45ebf573a63822bb9f9 100644
--- a/content/renderer/render_thread_impl.cc
+++ b/content/renderer/render_thread_impl.cc
@@ -117,6 +117,7 @@
 #include "content/renderer/renderer_blink_platform_impl.h"
 #include "content/renderer/scheduler/resource_dispatch_throttler.h"
 #include "content/renderer/service_worker/embedded_worker_dispatcher.h"
+#include "content/renderer/service_worker/embedded_worker_instance_client_impl.h"
 #include "content/renderer/service_worker/service_worker_context_client.h"
 #include "content/renderer/service_worker/service_worker_context_message_filter.h"
 #include "content/renderer/shared_worker/embedded_shared_worker_stub.h"
@@ -848,8 +849,11 @@ void RenderThreadImpl::Init(
   GetContentClient()->renderer()->ExposeInterfacesToBrowser(
       GetInterfaceRegistry());
 
-  GetInterfaceRegistry()->AddInterface(base::Bind(CreateFrameFactory));
-  GetInterfaceRegistry()->AddInterface(base::Bind(CreateEmbeddedWorkerSetup));
+  GetInterfaceRegistry()->AddInterface(base::Bind(&CreateFrameFactory));
+  GetInterfaceRegistry()->AddInterface(base::Bind(&CreateEmbeddedWorkerSetup));
+  GetInterfaceRegistry()->AddInterface(
+      base::Bind(&EmbeddedWorkerInstanceClientImpl::Create,
+                 base::Unretained(embedded_worker_dispatcher_.get())));

[horo, 2016/08/24 04:59:16]

I don't know well about the life cycle of InterfaceRegistry, but is it safe to
pass the unretained embedded_worker_dispatcher_?
If it is safe, please write comments about it.

[shimazu, 2016/08/25 05:26:16]

I thought this was no problem, but now I'm suspecting something wrong might
happen if an IPC request comes when executing RenderThreadImpl::Shutdown.

 
   GetRemoteInterfaces()->GetInterface(
       mojo::GetProxy(&storage_partition_service_));