Remove NOTIFICATION_EXTENSION_UNINSTALLED from extension_enable_flow.*

chrome/browser/ui/extensions/extension_enable_flow.cc

 // Copyright (c) 2012 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 "chrome/browser/ui/extensions/extension_enable_flow.h"
 
 #include "chrome/browser/chrome_notification_types.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser.h"
@@ skipping 98 matching lines @@
   prompt_.reset(parent_contents_ ?
       new ExtensionInstallPrompt(parent_contents_) :
       new ExtensionInstallPrompt(profile_, parent_window_, this));
 }
 
 void ExtensionEnableFlow::StartObserving() {
   extension_registry_observer_.Add(
       extensions::ExtensionRegistry::Get(profile_));
   registrar_.Add(this, chrome::NOTIFICATION_EXTENSION_LOAD_ERROR,
                  content::Source<Profile>(profile_));
-  registrar_.Add(this,
-                 chrome::NOTIFICATION_EXTENSION_UNINSTALLED_DEPRECATED,
-                 content::Source<Profile>(profile_));
 }
 
 void ExtensionEnableFlow::StopObserving() {
   registrar_.RemoveAll();
+  extension_registry_observer_.RemoveAll();

[limasdf, 2014/06/03 19:27:43]

I think this is necessary. and it should be addressed from my previous
commit(https://codereview.chromium.org/251603004/)

 }
 
 void ExtensionEnableFlow::Observe(int type,
                                   const content::NotificationSource& source,
                                   const content::NotificationDetails& details) {
-  switch (type) {
-    case chrome::NOTIFICATION_EXTENSION_LOAD_ERROR: {
-      StopObserving();
-      delegate_->ExtensionEnableFlowAborted(false);
-      break;
-    }
-    case chrome::NOTIFICATION_EXTENSION_UNINSTALLED_DEPRECATED: {
-      const Extension* extension =
-          content::Details<const Extension>(details).ptr();
-      if (extension->id() == extension_id_) {
-        StopObserving();
-        delegate_->ExtensionEnableFlowAborted(false);
-      }
-      break;
-    }
-    default:
-      NOTREACHED();
-  }
+  DCHECK_EQ(chrome::NOTIFICATION_EXTENSION_LOAD_ERROR, type);
+  StopObserving();
+  delegate_->ExtensionEnableFlowAborted(false);
 }
 
 void ExtensionEnableFlow::OnExtensionLoaded(
     content::BrowserContext* browser_context,
     const Extension* extension) {
   if (extension->id() == extension_id_) {
     StopObserving();
     CheckPermissionAndMaybePromptUser();
   }
 }
 
+void ExtensionEnableFlow::OnExtensionUninstalled(
+    content::BrowserContext* browser_context,
+    const Extension* extension) {
+  if (extension->id() == extension_id_) {
+    StopObserving();
+    delegate_->ExtensionEnableFlowAborted(false);
+  }
+}
+
 void ExtensionEnableFlow::InstallUIProceed() {
   ExtensionService* service =
       extensions::ExtensionSystem::Get(profile_)->extension_service();
 
   // The extension can be uninstalled in another window while the UI was
   // showing. Treat it as a cancellation and notify |delegate_|.
   const Extension* extension = service->GetExtensionById(extension_id_, true);
   if (!extension) {
     delegate_->ExtensionEnableFlowAborted(true);
     return;
   }
 
   service->GrantPermissionsAndEnableExtension(extension);
   delegate_->ExtensionEnableFlowFinished();  // |delegate_| may delete us.
 }
 
 void ExtensionEnableFlow::InstallUIAbort(bool user_initiated) {
   delegate_->ExtensionEnableFlowAborted(user_initiated);
   // |delegate_| may delete us.
 }
 
 content::WebContents* ExtensionEnableFlow::OpenURL(
     const content::OpenURLParams& params) {
   chrome::ScopedTabbedBrowserDisplayer displayer(
       profile_, chrome::GetActiveDesktop());
   return displayer.browser()->OpenURL(params);
 }