Create ServiceWorkerProviderHost before starting worker

content/common/service_worker/embedded_worker_start_params.h

Index: content/common/service_worker/embedded_worker_start_params.h
diff --git a/content/common/service_worker/embedded_worker_start_params.h b/content/common/service_worker/embedded_worker_start_params.h
index 0d432192bd315e82a6608abcf0b83d6b8ed8e98a..e2c26ce5703afa5d230fa5b6ce0fad47d61a2b3c 100644
--- a/content/common/service_worker/embedded_worker_start_params.h
+++ b/content/common/service_worker/embedded_worker_start_params.h
@@ -11,6 +11,8 @@
 
 namespace content {
 
+class ServiceWorkerProviderHost;
+
 struct CONTENT_EXPORT EmbeddedWorkerStartParams {
   EmbeddedWorkerStartParams();
 
@@ -23,6 +25,7 @@ struct CONTENT_EXPORT EmbeddedWorkerStartParams {
   bool wait_for_debugger;
   bool is_installed;
   EmbeddedWorkerSettings settings;
+  ServiceWorkerProviderHost* controller_provider;

[falken, 2017/06/12 07:01:28]

It seems weird that content/common has a message with ServiceWorkerProviderHost
in it, since that's a browser process concept.

Does controller_provider need to live in this struct?

[shimazu, 2017/06/14 07:24:40]

Actually it was because I could avoid lots of changes in unittests. It's not
necessary to be owned by StartParams, but I originally thought
ServiceWorkerProvider's info could be an parameter of it.

 };
 
 }  // namespace content