Manage ServiceWorkerDispatcherHost in ServiceWorkerContextCore

content/common/service_worker/service_worker_provider.mojom

Index: content/common/service_worker/service_worker_provider.mojom
diff --git a/content/common/service_worker/service_worker_provider.mojom b/content/common/service_worker/service_worker_provider.mojom
new file mode 100644
index 0000000000000000000000000000000000000000..d28c391389b65114454498aaaa0de03a5b9a2e4d
--- /dev/null
+++ b/content/common/service_worker/service_worker_provider.mojom
@@ -0,0 +1,32 @@
+// 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;
+
+import "content/common/service_worker/service_worker_types.mojom";
+
+// A container object carried from the renderer to the browser process.
+// This contains the params for the constructor of ServiceWorkerProviderHost.
+//
+// |provider_id| is unique within its child process. When this provider is
+// created for a document, |route_id| is the frame ID of it. When this provider
+// is created for a Shared Worker, |route_id| is the Shared Worker route
+// ID. When this provider is created for a Service Worker, |route_id| is
+// MSG_ROUTING_NONE. |provider_type| identifies whether this provider is for
+// Service Worker controllees (documents and Shared Workers) or for controllers
+// (Service Workers). |is_parent_frame_secure| is false if the provider is
+// created for a document whose parent frame is not secure from the point of
+// view of the document; that is, blink::WebFrame::canHaveSecureChild() returns
+// false. This doesn't mean the document is necessarily an insecure context,
+// because the document may have a URL whose scheme is granted an exception that
+// allows bypassing the ancestor secure context check. See the comment in
+// blink::Document::isSecureContextImpl for more details. If the provider is not
+// created for a document, or the document does not have a parent frame,
+// |is_parent_frame_secure| is true.

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

I slightly prefer these comments to be in
	content/common/service_worker/service_worker_provider.h since it's hard to find
.mojom files in codesearch. Either way, one should link to the other.

Also, seems easier to read if the comments are directly in front of the fields,
instead of all at the top.

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

Done.

+struct ServiceWorkerProviderHostInfo {
+  int32 provider_id;
+  int32 route_id;
+  ServiceWorkerProviderType type;
+  bool is_parent_frame_secure;
+};