Create ServiceWorkerProviderHost before starting worker

content/common/service_worker/embedded_worker_start_params.h

 // 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.
 
 #ifndef CONTENT_COMMON_SERVICE_WORKER_EMBEDDED_WORKER_START_PARAMS_H_
 #define CONTENT_COMMON_SERVICE_WORKER_EMBEDDED_WORKER_START_PARAMS_H_
 
 #include "content/common/content_export.h"
 #include "content/common/service_worker/embedded_worker_settings.h"
 #include "url/gurl.h"
 
 namespace content {
 
+class ServiceWorkerProviderHost;
+
 struct CONTENT_EXPORT EmbeddedWorkerStartParams {
   EmbeddedWorkerStartParams();
 
   int embedded_worker_id;
   int64_t service_worker_version_id;
   GURL scope;
   GURL script_url;
   int worker_devtools_agent_route_id;
   bool pause_after_download;
   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
 
 #endif  // CONTENT_COMMON_SERVICE_WORKER_EMBEDDED_WORKER_START_PARAMS_H_