Service Worker: set active worker to REDUNDANT when unregistered

content/browser/service_worker/service_worker_unregister_job.cc

Index: content/browser/service_worker/service_worker_unregister_job.cc
diff --git a/content/browser/service_worker/service_worker_unregister_job.cc b/content/browser/service_worker/service_worker_unregister_job.cc
index f60a3b99d3d6cd48880d515b7cde73baf5a77034..7cfad3e1c9b95be06bafc53c6edf6281f1550bdd 100644
--- a/content/browser/service_worker/service_worker_unregister_job.cc
+++ b/content/browser/service_worker/service_worker_unregister_job.cc
@@ -71,11 +71,6 @@ void ServiceWorkerUnregisterJob::DeleteRegistration(
   // TODO(nhiroki): When we've implemented the installing version, terminate and
   // set it to redundant here as per spec.
 
-  if (registration->waiting_version()) {
-    registration->waiting_version()->SetStatus(
-        ServiceWorkerVersion::REDUNDANT);

[michaeln, 2014/07/10 00:55:23]

i think we have to Doom the active version if there is a waiting version. as
coded, the storage class will Doom at most one version per registration.

[falken, 2014/07/10 14:05:12]

I thought this would work because when an updated version boots out the existing
one (waiting gets stored, active gets deleted), the storage class would wait for
the NoControllees signal from the active and doom it then.

-  }
-
   context_->storage()->DeleteRegistration(
       registration->id(),
       registration->script_url().GetOrigin(),