Manage ServiceWorkerDispatcherHost in ServiceWorkerContextCore

content/common/service_worker/service_worker.mojom

Index: content/common/service_worker/service_worker.mojom
diff --git a/content/common/service_worker/service_worker.mojom b/content/common/service_worker/service_worker.mojom
index 840a3efdc575d57a070b4347144b7eab987133c7..04f59d21128e9bdcddb9ea766c2d26b3425d8fe4 100644
--- a/content/common/service_worker/service_worker.mojom
+++ b/content/common/service_worker/service_worker.mojom
@@ -4,18 +4,19 @@
 
 module content.mojom;
 
-import "content/common/service_worker/service_worker_types.mojom";
+import "content/common/service_worker/service_worker_provider.mojom";
 
 // Per-process browser-side interface bound to ServiceWorkerDispatcherHost.
 // Each InterfacePtrs on the same render process will be bound to the same
 // ServiceWorkerDispatcherHost.
 interface ServiceWorkerDispatcherHost {
-  OnProviderCreated(int32 provider_id,
-                    int32 route_id,
-                    ServiceWorkerProviderType provider_type,
-                    bool is_parent_frame_secure);
+  OnProviderCreated(ServiceWorkerProviderHostInfo provider_info);
   OnProviderDestroyed(int32 provider_id);
-  // Make relationship between the network provider and the service worker
+
+  // Informs the browser that a service worker is starting up in a provider.

[falken, 2017/02/08 06:53:51]

nit: let's just remove "in a provider". A SW can only start up in a provider.

[shimazu, 2017/02/13 03:25:56]

Done.

+  // |provider_id| identifies the ServiceWorkerProviderHost hosting the service
+  // worker. |version_id| identifies the ServiceWorkerVersion and
+  // |embedded_worker_id| identifies the EmbeddedWorkerInstance.
   OnSetHostedVersionId(int32 provider_id,
                        int64 version_id,
                        int32 embedded_worker_id);