Start EmbeddedWorker during registration - take 2

content/browser/service_worker/service_worker_context_core.h

Index: content/browser/service_worker/service_worker_context_core.h
diff --git a/content/browser/service_worker/service_worker_context_core.h b/content/browser/service_worker/service_worker_context_core.h
index c59ef7cbb1d3fe5ba5bae3a51e98f98d2c967039..d90f0c2a75b89ee40e4047f6840fe353fada3318 100644
--- a/content/browser/service_worker/service_worker_context_core.h
+++ b/content/browser/service_worker/service_worker_context_core.h
@@ -70,10 +70,12 @@ class CONTENT_EXPORT ServiceWorkerContextCore
   // The callback will be called on the IO thread.
   void RegisterServiceWorker(const GURL& pattern,
                              const GURL& script_url,
+                             int source_process_id,

[kinuko, 2014/01/29 09:56:20]

I'm not fully sure yet if passing process_id here is the right thing, but during
the registration flow it'd be probably most convenient (at least for now..)

[alecflett, 2014/01/30 01:51:13]

Yeah - I mean at least passing the ServiceWorkerProviderHost gave a hint as for
what this is for.. but for reduced dependencies I think process id will work
here.

                              const RegistrationCallback& callback);
 
   // The callback will be called on the IO thread.
   void UnregisterServiceWorker(const GURL& pattern,
+                               int source_process_id,
                                const UnregistrationCallback& callback);
 
   EmbeddedWorkerRegistry* embedded_worker_registry() {
@@ -100,8 +102,8 @@ class CONTENT_EXPORT ServiceWorkerContextCore
 
   ProcessToProviderMap providers_;
   scoped_ptr<ServiceWorkerStorage> storage_;
-  scoped_ptr<ServiceWorkerJobCoordinator> job_coordinator_;
   scoped_refptr<EmbeddedWorkerRegistry> embedded_worker_registry_;
+  scoped_ptr<ServiceWorkerJobCoordinator> job_coordinator_;
 
   DISALLOW_COPY_AND_ASSIGN(ServiceWorkerContextCore);
 };