Service Worker: Make ServiceWorkerRegistration.update() return a promise. (Chromium 2/3)

content/browser/service_worker/service_worker_register_job.h

Index: content/browser/service_worker/service_worker_register_job.h
diff --git a/content/browser/service_worker/service_worker_register_job.h b/content/browser/service_worker/service_worker_register_job.h
index 537ac4cbae112effc6a3c308334d2ffae33de9d0..124cefdf45dbd6aba4725c7f52f649d8832fe5cb 100644
--- a/content/browser/service_worker/service_worker_register_job.h
+++ b/content/browser/service_worker/service_worker_register_job.h
@@ -55,8 +55,8 @@ class ServiceWorkerRegisterJob : public ServiceWorkerRegisterJobBase {
 
   // Registers a callback to be called when the promise would resolve (whether
   // successfully or not). Multiple callbacks may be registered.
-  // If |provider_host| is not NULL, its process will be regarded as a candidate
-  // process to run the worker.
+  // If |provider_host| is not NULL, its process will be regarded as a
+  // candidate process to run the worker.

[nhiroki, 2015/07/31 06:49:11]

nit: This change looks not necessary.

[jungkees, 2015/07/31 07:09:58]

Yes, right. This was changed when I tried to make |provider_host| to
|opt_provider_host| but I rather have done it by overloading the functions after
all. I'll leave the original comment as-is.

   void AddCallback(const RegistrationCallback& callback,
                    ServiceWorkerProviderHost* provider_host);