Cleanup geofence registrations when a service worker is unregistered.

content/browser/service_worker/service_worker_context_core.cc

 // Copyright 2013 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.
 
 #include "content/browser/service_worker/service_worker_context_core.h"
 
 #include "base/barrier_closure.h"
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/files/file_path.h"
@@ skipping 292 matching lines @@
   callback.Run(status, registration->id());
   if (observer_list_.get()) {
     observer_list_->Notify(&ServiceWorkerContextObserver::OnRegistrationStored,
                            pattern);
   }
 }
 
 void ServiceWorkerContextCore::UnregistrationComplete(
     const GURL& pattern,
     const ServiceWorkerContextCore::UnregistrationCallback& callback,
+    int64 registration_id,
     ServiceWorkerStatusCode status) {
   callback.Run(status);
   if (observer_list_.get()) {
     observer_list_->Notify(&ServiceWorkerContextObserver::OnRegistrationDeleted,

[michaeln, 2015/01/09 22:58:41]

Seems odd that this notif can happen even if nothing was deleted (unlike the
Stored notif)? In that case the ID kInvalidServiceWorkerRegistrationId.

[Marijn Kruisselbrink, 2015/01/09 23:13:15]

Yes, that does seem odd (but not anything introduced by my change; at least not
it will be possible for an observer to detect this cast). Should I file a bug
for this?

-                           pattern);
+                           registration_id, pattern);
   }
 }
 
 ServiceWorkerRegistration* ServiceWorkerContextCore::GetLiveRegistration(
     int64 id) {
   RegistrationsMap::iterator it = live_registrations_.find(id);
   return (it != live_registrations_.end()) ? it->second : NULL;
 }
 
 void ServiceWorkerContextCore::AddLiveRegistration(
@@ skipping 167 matching lines @@
           source_identifier, message_level, message, line_number, source_url));
 }
 
 ServiceWorkerProcessManager* ServiceWorkerContextCore::process_manager() {
   if (wrapper_)
     return wrapper_->process_manager();
   return NULL;
 }
 
 }  // namespace content