[ServiceWorker] Check is_uninstalled in ServiceWorkerRegisterJob::ContinueWithRegistration()

content/browser/service_worker/service_worker_registration.cc

Index: content/browser/service_worker/service_worker_registration.cc
diff --git a/content/browser/service_worker/service_worker_registration.cc b/content/browser/service_worker/service_worker_registration.cc
index 33fdfa6767fee298fafa53e31041cca2d786cf80..ba3298cea61129e982a1aa9914e674f22ae01813 100644
--- a/content/browser/service_worker/service_worker_registration.cc
+++ b/content/browser/service_worker/service_worker_registration.cc
@@ -30,6 +30,7 @@ ServiceWorkerRegistration::ServiceWorkerRegistration(
       registration_id_(registration_id),
       is_deleted_(false),
       is_uninstalling_(false),
+      is_uninstalled_(false),
       should_activate_when_ready_(false),
       context_(context) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
@@ -315,6 +316,7 @@ void ServiceWorkerRegistration::Clear() {
 
   FOR_EACH_OBSERVER(
       Listener, listeners_, OnRegistrationFinishedUninstalling(this));
+  is_uninstalled_ = true;

[michaeln, 2014/09/11 19:31:40]

Would it make sense to set this flag immediately after line 290? Between there
and here, a whole lot of observer methods are called and they'll see a reg that
is not uninstalling and also not uninstalled. Thats inconsistent with the new
found way to represent is_uninstalled.

[horo, 2014/09/12 01:09:25]

That's make sense.
Done.

Thank you.

 }
 
 void ServiceWorkerRegistration::OnRestoreFinished(