Add two interfaces for ServiceWorkerProviderContext/ProviderHost

content/common/service_worker/service_worker.mojom

 // Copyright 2016 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.
 
 module content.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.
+// Per-process browser-side interface bound to
+// content::ServiceWorkerDispatcherHost.
+// All InterfacePtrs on the same render process are bound to the same
+// content::ServiceWorkerDispatcherHost. The renderer uses this interface to
+// tell the browser when potential service worker clients are created and when
+// service workers are starting up.

[falken, 2017/05/30 04:12:32]

I guess this violates our new policy of not mentioning impl specifically in the
interface comments.

But the comment already existed before this CL no need to block on it. We can
work this later.

[shimazu, 2017/05/30 07:24:35]

Acknowledged.

[shimazu, 2017/06/01 08:18:54]

Addressed this at https://chromium-review.googlesource.com/520445

 interface ServiceWorkerDispatcherHost {
+  // Creates a content::ServiceWorkerProviderHost on the browser
+  // process. |provider_info| has Mojo endpoints to connect the provider host
+  // and the provider on the renderer together. The lifetime of
+  // ServiceWorkerProviderHost will be tied to the
+  // mojom::ServiceWorkerProviderHost interface.
   OnProviderCreated(ServiceWorkerProviderHostInfo provider_info);
-  OnProviderDestroyed(int32 provider_id);
 
   // Informs the browser that a service worker is starting up. |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);
 };