Add two interfaces for ServiceWorkerProviderContext/ProviderHost

content/common/service_worker/service_worker_provider_interfaces.mojom

+// Copyright 2017 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;
+
+// mojom::ServiceWorkerProviderHost is a browser-side
+// interface bound to content::ServiceWorkerProviderHost. The renderer uses this

[kinuko, 2017/05/23 12:57:20]

Hmm, I can see what this comment tries to clarify but to me it feels a bit weird
to explicitly note that some interface is bound to some implementation... this
is a definition for an interface that can be implemented by any impl class,
while content::ServiceWorkerProviderHost is one of the implementation class,
right?

[falken, 2017/05/29 08:47:28]

Fair enough, yea. Let's remove the mentions of the implementation. Does it still
make sense to call this a "browser-side interface"? Seems like the common case
even though I guess the impl could put the interface on the renderer.

[shimazu, 2017/05/30 03:50:07]

Thanks for your comments. It sounds good to separate the implementation detail
out from mojom. Done.

+// interface to request the browser to do operations involving service worker
+// registrations.
+// A content::ServiceWorkerProviderHost bound with this interface will be destroyed when
+// the connection is disconnected.

[kinuko, 2017/05/23 12:57:20]

Again this seems to be implementation detail which is unrelated to this
interface definition.

[falken, 2017/05/29 08:47:28]

Sure let's move this documentation to content::ServiceWorkerProviderHost. Class
documentation should talk about the class's lifetime.

[shimazu, 2017/05/30 03:50:07]

Done.

+interface ServiceWorkerProviderHost {
+  // TODO(shimazu): implement them.
+  // register() => ();
+  // getRegistation() => ();
+  // getRegistrations() => ();
+  // getRegistrationForReady() => ();
+};
+
+// mojom::ServiceWorkerProvider is a renderer-side interface bound to
+// content::ServiceWorkerProviderContext.

[falken, 2017/05/29 08:47:28]

By the same logic I guess we should just call this a renderer-side interface.
"The browser process uses this interface to do XXX"?

[shimazu, 2017/05/30 03:50:07]

Done.

+interface ServiceWorkerProvider {
+  // TODO(shimazu): implement them.
+  // associateRegistration();
+  // disassociateRegistration();
+  // setControllerServiceWorker();
+  // messageToDocument();
+};