ServiceWorker: Call SyncMatchingRegistration when document_url is changed

content/browser/service_worker/service_worker_provider_host.h

Index: content/browser/service_worker/service_worker_provider_host.h
diff --git a/content/browser/service_worker/service_worker_provider_host.h b/content/browser/service_worker/service_worker_provider_host.h
index 0ca74533ce6542014a60acf86db9e28ffc29aace..71d51f8c71581dd0b37a7cac8fed6ffef35d6496 100644
--- a/content/browser/service_worker/service_worker_provider_host.h
+++ b/content/browser/service_worker/service_worker_provider_host.h
@@ -137,6 +137,9 @@ class CONTENT_EXPORT ServiceWorkerProviderHost
     return running_hosted_version_.get();
   }
 
+  // SetDocumentUrl sets the |document_url_|.  When this object is for a client,

[nhiroki, 2016/08/22 12:41:37]

"SetDocumentUrl" looks redundant. "Sets the |document_url_|." would be
sufficient.

[shimazu, 2016/08/23 01:53:52]

Done.

+  // |matching_registrations_| gets also updated to ensure that |document_url_|
+  // is in scope of all |matching_registrations_|.
   void SetDocumentUrl(const GURL& url);
   const GURL& document_url() const { return document_url_; }
 
@@ -246,9 +249,6 @@ class CONTENT_EXPORT ServiceWorkerProviderHost
   void AddMatchingRegistration(ServiceWorkerRegistration* registration);
   void RemoveMatchingRegistration(ServiceWorkerRegistration* registration);
 
-  // Add matched registrations for document generated by shift-reload.
-  void AddAllMatchingRegistrations();
-
   // An optimized implementation of [[Match Service Worker Registration]]
   // for current document.
   ServiceWorkerRegistration* MatchRegistration() const;
@@ -311,6 +311,12 @@ class CONTENT_EXPORT ServiceWorkerProviderHost
 
   void SendAssociateRegistrationMessage();
 
+  // Syncs matching registrations with live registrations

[nhiroki, 2016/08/22 12:41:37]

Missing trailing dot.

[shimazu, 2016/08/23 01:53:52]

Done.

+  void SyncMatchingRegistrations();
+
+  // Discards all references to matching registrations

[nhiroki, 2016/08/22 12:41:37]

ditto.

[shimazu, 2016/08/23 01:53:52]

Done.

+  void RemoveAllMatchingRegistrations();
+
   // Increase/decrease this host's process reference for |pattern|.
   void IncreaseProcessReference(const GURL& pattern);
   void DecreaseProcessReference(const GURL& pattern);