[ServiceWorker] Mojofy InstallEvent of Service Worker

content/renderer/service_worker/service_worker_context_client.h

Index: content/renderer/service_worker/service_worker_context_client.h
diff --git a/content/renderer/service_worker/service_worker_context_client.h b/content/renderer/service_worker/service_worker_context_client.h
index 5385b98d16a2cb5168cacccff78f17eb17033c4b..9c01bac803dc9bd1e42472cee4b032e4e7857aa6 100644
--- a/content/renderer/service_worker/service_worker_context_client.h
+++ b/content/renderer/service_worker/service_worker_context_client.h
@@ -224,6 +224,9 @@ class ServiceWorkerContextClient : public blink::WebServiceWorkerContextClient,
       const ServiceWorkerVersionAttributes& attrs);
 
   // mojom::ServiceWorkerEventDispatcher
+  void DispatchInstallEvent(
+      mojom::InstallEventMethodAssociatedPtrInfo client,
+      const DispatchInstallEventCallback& callback) override;
   void DispatchActivateEvent(
       const DispatchActivateEventCallback& callback) override;
   void DispatchBackgroundFetchAbortEvent(
@@ -271,7 +274,6 @@ class ServiceWorkerContextClient : public blink::WebServiceWorkerContextClient,
       const DispatchPaymentRequestEventCallback& callback) override;
   void Ping(const PingCallback& callback) override;
 
-  void OnInstallEvent(int request_id);
   void OnNotificationClickEvent(
       int request_id,
       const std::string& notification_id,
@@ -332,6 +334,9 @@ class ServiceWorkerContextClient : public blink::WebServiceWorkerContextClient,
   // This is bound on the worker thread.
   mojom::ServiceWorkerEventDispatcherRequest pending_dispatcher_request_;
 
+  // The destination for the install event method.
+  mojom::InstallEventMethodAssociatedPtr install_method_client_;

[shimazu, 2017/04/05 05:01:27]

I think you are trying to have the method interface for each event. In this
case, each interface pointers passed through DispatchSomeEvent should be kept by
each event instead of directly owned by SWContextClient. I mean it should be
mapped to each event id. For implementation, you might add the event_id to the
ctor of InstallEvent in blink to return the id back to SWContextClient when
registerForeignFetchScopes is called.

+
   // Renderer-side object corresponding to WebEmbeddedWorkerInstance.
   // This is valid from the ctor to workerContextDestroyed.
   std::unique_ptr<EmbeddedWorkerInstanceClientImpl> embedded_worker_client_;