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 |
index 4016e4266254d6403c1afc96329018930ed3b932..66afc9b6a71881fd22ee843073bb7f33fc8efa16 100644 |
--- a/content/common/service_worker/service_worker_provider.mojom |
+++ b/content/common/service_worker/service_worker_provider.mojom |
@@ -7,6 +7,22 @@ module content.mojom; |
import "content/common/service_worker/service_worker_provider_interfaces.mojom"; |
import "content/common/service_worker/service_worker_types.mojom"; |
+// A container object carried from the browser to the renderer process. |
+// This contains the params for the constructor of ServiceWorkerNetworkProvider. |
+// This is passed on launching a service worker. |
+struct ServiceWorkerProviderClientInfo { |
falken
2017/06/16 15:44:47
Can you help me understand this terminology?
A Se
shimazu
2017/06/19 03:53:30
I originally thought that "ServiceWorkerProviderCl
falken
2017/06/19 05:21:43
These are great questions. I'm hoping kinuko@ or m
shimazu
2017/06/19 06:00:39
Thanks, ServiceWorkerProviderInfoForStartWorker so
|
+ int32 provider_id; |
+ ServiceWorkerProviderType type; |
+ bool is_parent_frame_secure; |
falken
2017/06/16 15:44:46
I don't think |type| and |is_parent_frame_secure|
shimazu
2017/06/19 03:53:30
Yes, that's right.
Removed.
|
+ // |registration| and |attributes| are information about the registration |
+ // currently to be launched. |
falken
2017/06/16 15:44:47
This terminology is a little inaccurate because th
falken
2017/06/16 15:56:30
And add "used to populate ServiceWorkerGlobalScope
shimazu
2017/06/19 03:53:30
Done.
|
+ ServiceWorkerRegistrationObjectInfo registration; |
+ ServiceWorkerVersionAttributes attributes; |
+ |
+ associated ServiceWorkerProviderHost host_ptr_info; |
+ associated ServiceWorkerProvider& client_request; |
+}; |
+ |
// A container object carried from the renderer to the browser process. |
// This contains the parameters to specify the provider on the browser side. |
// See also comments in |