Change extension unload notification to indicate updates....

chrome/browser/background_application_list_model.cc

 // Copyright (c) 2010 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/background_application_list_model.h"
 
 #include <algorithm>
 #include <set>
 
 #include "app/l10n_util_collator.h"
@@ skipping 135 matching lines @@
 BackgroundApplicationListModel::BackgroundApplicationListModel(Profile* profile)
     : profile_(profile) {
   DCHECK(profile_);
   registrar_.Add(this,
                  NotificationType::EXTENSION_LOADED,
                  Source<Profile>(profile));
   registrar_.Add(this,
                  NotificationType::EXTENSION_UNLOADED,
                  Source<Profile>(profile));
   registrar_.Add(this,
-                 NotificationType::EXTENSION_UNLOADED_DISABLED,
-                 Source<Profile>(profile));
-  registrar_.Add(this,
                  NotificationType::EXTENSIONS_READY,
                  Source<Profile>(profile));
   ExtensionService* service = profile->GetExtensionService();
   if (service && service->is_ready())
     Update();
 }
 
 void BackgroundApplicationListModel::AddObserver(Observer* observer) {
   observers_.AddObserver(observer);
 }
@@ skipping 85 matching lines @@
   }
   ExtensionService* service = profile_->GetExtensionService();
   if (!service || !service->is_ready())
     return;
 
   switch (type.value) {
     case NotificationType::EXTENSION_LOADED:
       OnExtensionLoaded(Details<Extension>(details).ptr());
       break;
     case NotificationType::EXTENSION_UNLOADED:
-      // Handle extension unload uniformly, falling through to next case.
-    case NotificationType::EXTENSION_UNLOADED_DISABLED:
-      OnExtensionUnloaded(Details<Extension>(details).ptr());
+      OnExtensionUnloaded(Details<UnloadedExtensionInfo>(details)->extension);
       break;
     default:
       NOTREACHED() << "Received unexpected notification";
   }
 }
 
 void BackgroundApplicationListModel::OnApplicationDataChanged(
     const Extension* extension) {
   FOR_EACH_OBSERVER(Observer, observers_, OnApplicationDataChanged(extension));
 }
 
 void BackgroundApplicationListModel::OnExtensionLoaded(Extension* extension) {
   // We only care about extensions that are background applications
   if (!IsBackgroundApp(*extension))
     return;
   AssociateApplicationData(extension);
   Update();
 }
 
-void BackgroundApplicationListModel::OnExtensionUnloaded(Extension* extension) {
+void BackgroundApplicationListModel::OnExtensionUnloaded(
+    const Extension* extension) {
   if (!IsBackgroundApp(*extension))
     return;
   Update();
   DissociateApplicationData(extension);
 }
 
 void BackgroundApplicationListModel::RemoveObserver(Observer* observer) {
   observers_.RemoveObserver(observer);
 }
 
@@ skipping 15 matching lines @@
          (*old_cursor)->name() == (*new_cursor)->name() &&
          (*old_cursor)->id() == (*new_cursor)->id()) {
     ++old_cursor;
     ++new_cursor;
   }
   if (old_cursor != extensions_.end() || new_cursor != extensions.end()) {
     extensions_ = extensions;
     FOR_EACH_OBSERVER(Observer, observers_, OnApplicationListChanged());
   }
 }