ServiceWorker: propagate provider_id to the browser process in .register + int32/int cleanup

content/browser/service_worker/service_worker_context_core.cc

Index: content/browser/service_worker/service_worker_context_core.cc
diff --git a/content/browser/service_worker/service_worker_context_core.cc b/content/browser/service_worker/service_worker_context_core.cc
index e6dc32f23807b94d0734c35b0110e0d0b4ebc4e0..2d2e830de81ed227f277e6f9d6d0d0d14d54887f 100644
--- a/content/browser/service_worker/service_worker_context_core.cc
+++ b/content/browser/service_worker/service_worker_context_core.cc
@@ -69,9 +69,13 @@ void ServiceWorkerContextCore::RegisterServiceWorker(
     const GURL& pattern,
     const GURL& script_url,
     int source_process_id,
+    ServiceWorkerProviderHost* provider_host,
     const RegistrationCallback& callback) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
 
+  // TODO(kinuko): Wire the provider_host so that we can tell which document
+  // is calling .register.
+
   job_coordinator_->Register(
       pattern,
       script_url,
@@ -84,9 +88,13 @@ void ServiceWorkerContextCore::RegisterServiceWorker(
 void ServiceWorkerContextCore::UnregisterServiceWorker(
     const GURL& pattern,
     int source_process_id,
+    ServiceWorkerProviderHost* provider_host,

[Jeffrey Yasskin, 2014/04/11 23:21:24]

Note that ServiceWorkerJobCoordinator::Unregister doesn't currently use the
source_process_id: 
https://code.google.com/p/chromium/codesearch/#chromium/src/content/browser/s...
Could you double-check that it's actually going to use the ProviderHost?

     const UnregistrationCallback& callback) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
 
+  // TODO(kinuko): Wire the provider_host so that we can tell which document
+  // is calling .register.
+
   job_coordinator_->Unregister(pattern, source_process_id, callback);
 }