Make InstallTracker use more appropriate names for events.
BUG=247555
Review URL: https://chromiumcodereview.appspot.com/18226006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@211328 0039d316-1c4b-4281-b951-d872f2087c98
diff --git a/chrome/browser/extensions/install_tracker.cc b/chrome/browser/extensions/install_tracker.cc
index 784c97c..ea0e6aa 100644
--- a/chrome/browser/extensions/install_tracker.cc
+++ b/chrome/browser/extensions/install_tracker.cc
@@ -16,6 +16,8 @@
extensions::ExtensionPrefs* prefs) {
ExtensionSorting* sorting = prefs->extension_sorting();
+ registrar_.Add(this, chrome::NOTIFICATION_EXTENSION_INSTALLED,
+ content::Source<Profile>(profile));
registrar_.Add(this, chrome::NOTIFICATION_EXTENSION_LOADED,
content::Source<Profile>(profile));
registrar_.Add(this, chrome::NOTIFICATION_EXTENSION_UNLOADED,
@@ -78,24 +80,26 @@
const content::NotificationSource& source,
const content::NotificationDetails& details) {
switch (type) {
- case chrome::NOTIFICATION_EXTENSION_LOADED: {
+ case chrome::NOTIFICATION_EXTENSION_INSTALLED: {
const Extension* extension =
content::Details<const Extension>(details).ptr();
FOR_EACH_OBSERVER(InstallObserver, observers_,
OnExtensionInstalled(extension));
break;
}
+ case chrome::NOTIFICATION_EXTENSION_LOADED: {
+ const Extension* extension =
+ content::Details<const Extension>(details).ptr();
+ FOR_EACH_OBSERVER(InstallObserver, observers_,
+ OnExtensionLoaded(extension));
+ break;
+ }
case chrome::NOTIFICATION_EXTENSION_UNLOADED: {
const content::Details<extensions::UnloadedExtensionInfo>& unload_info(
details);
const Extension* extension = unload_info->extension;
- if (unload_info->reason == extension_misc::UNLOAD_REASON_UNINSTALL) {
- FOR_EACH_OBSERVER(InstallObserver, observers_,
- OnExtensionUninstalled(extension));
- } else {
- FOR_EACH_OBSERVER(InstallObserver, observers_,
- OnExtensionDisabled(extension));
- }
+ FOR_EACH_OBSERVER(InstallObserver, observers_,
+ OnExtensionUnloaded(extension));
break;
}
case chrome::NOTIFICATION_EXTENSION_UNINSTALLED: {
