ServiceWorker: Mojofication of EWInstance::StopWorker

content/renderer/service_worker/embedded_worker_instance_client_impl.h

Index: content/renderer/service_worker/embedded_worker_instance_client_impl.h
diff --git a/content/renderer/service_worker/embedded_worker_instance_client_impl.h b/content/renderer/service_worker/embedded_worker_instance_client_impl.h
index f05c63ab4915d9a3ab880071e209a4311b1ec3f6..64d163fe24886f1686f58118c265f9b5c3a1ccf7 100644
--- a/content/renderer/service_worker/embedded_worker_instance_client_impl.h
+++ b/content/renderer/service_worker/embedded_worker_instance_client_impl.h
@@ -6,9 +6,10 @@
 #define CONTENT_RENDERER_SERVICE_WORKER_EMBEDDED_WORKER_REGISTRY_CLIENT_IMPL_H_
 
 #include "base/id_map.h"
+#include "base/optional.h"
 #include "content/common/service_worker/embedded_worker.mojom.h"
 #include "content/renderer/service_worker/embedded_worker_dispatcher.h"
-#include "mojo/public/cpp/bindings/strong_binding.h"
+#include "mojo/public/cpp/bindings/binding.h"
 
 namespace content {
 
@@ -19,22 +20,36 @@ class EmbeddedWorkerInstanceClientImpl
       EmbeddedWorkerDispatcher* dispatcher,
       mojo::InterfaceRequest<mojom::EmbeddedWorkerInstanceClient> request);
 
-  // Implementation of mojo interface
-  void StartWorker(mojom::EmbeddedWorkerStartWorkerParamsPtr params) override;
-
- private:
   EmbeddedWorkerInstanceClientImpl(
       EmbeddedWorkerDispatcher* dispatcher,
       mojo::InterfaceRequest<mojom::EmbeddedWorkerInstanceClient> request);
   ~EmbeddedWorkerInstanceClientImpl() override;
 
+  // Called from ServiceWorkerContextClient. Must call on the main thread.
+  void StopWorkerCompleted();
+
+ private:
+  // Implementation of mojo interface
+  void StartWorker(mojom::EmbeddedWorkerStartWorkerParamsPtr params) override;
+  void StopWorker(const StopWorkerCallback& callback) override;
+
+  // Handler of connection error bound to |binding_|
   void OnError();
 
   EmbeddedWorkerDispatcher* dispatcher_;
-  mojo::StrongBinding<mojom::EmbeddedWorkerInstanceClient> binding_;
+  // This object will be bound to the main thread.
+  mojo::Binding<mojom::EmbeddedWorkerInstanceClient> binding_;
 
+  // This is valid before StartWorker is called. After that, this object
+  // will be passed to ServiceWorkerContextClient.
+  std::unique_ptr<EmbeddedWorkerInstanceClientImpl> temporal_self_;
+
+  base::Optional<int> embedded_worker_id_;
   EmbeddedWorkerDispatcher::WorkerWrapper* wrapper_;
 
+  // Stores callbacks
+  StopWorkerCallback stop_callback_;
+
   DISALLOW_COPY_AND_ASSIGN(EmbeddedWorkerInstanceClientImpl);
 };