Kill NOTIFICATION_EXTENSION_UNLOADED_DEPRECATED
There's no place where NOTIFICATION_EXTENSION_UNLOADED_DEPRECATED is observed.
BUG=411568
Review-Url: https://codereview.chromium.org/2824673002
Cr-Commit-Position: refs/heads/master@{#466951}
diff --git a/chrome/browser/background/background_contents_service.cc b/chrome/browser/background/background_contents_service.cc
index eac5c9bf..9a30c89 100644
--- a/chrome/browser/background/background_contents_service.cc
+++ b/chrome/browser/background/background_contents_service.cc
@@ -177,7 +177,7 @@
// Origin URL must be different from the crashed extension to avoid the
// conflict. NotificationSystemObserver will cancel all notifications from
- // the same origin when NOTIFICATION_EXTENSION_UNLOADED_DEPRECATED.
+ // the same origin when OnExtensionUnloaded() is called.
Notification notification(message_center::NOTIFICATION_TYPE_SIMPLE,
base::string16(),
message,
diff --git a/chrome/browser/extensions/extension_service.cc b/chrome/browser/extensions/extension_service.cc
index 42205f6..7f8b30c 100644
--- a/chrome/browser/extensions/extension_service.cc
+++ b/chrome/browser/extensions/extension_service.cc
@@ -1172,15 +1172,8 @@
void ExtensionService::NotifyExtensionUnloaded(
const Extension* extension,
UnloadedExtensionInfo::Reason reason) {
- UnloadedExtensionInfo details(extension, reason);
-
registry_->TriggerOnUnloaded(extension, reason);
- content::NotificationService::current()->Notify(
- extensions::NOTIFICATION_EXTENSION_UNLOADED_DEPRECATED,
- content::Source<Profile>(profile_),
- content::Details<UnloadedExtensionInfo>(&details));
-
renderer_helper_->OnExtensionUnloaded(*extension);
system_->UnregisterExtensionWithRequestContexts(extension->id(), reason);
diff --git a/chrome/browser/themes/theme_service.cc b/chrome/browser/themes/theme_service.cc
index 402ab6e..425d762 100644
--- a/chrome/browser/themes/theme_service.cc
+++ b/chrome/browser/themes/theme_service.cc
@@ -547,7 +547,7 @@
// There should be no more infobars. This may not be the case because of
// http://crbug.com/62154
// RemoveUnusedThemes is called on a task because ClearAllThemeData() may
- // be called as a result of NOTIFICATION_EXTENSION_UNLOADED_DEPRECATED.
+ // be called as a result of OnExtensionUnloaded().
base::ThreadTaskRunnerHandle::Get()->PostTask(
FROM_HERE, base::Bind(&ThemeService::RemoveUnusedThemes,
weak_ptr_factory_.GetWeakPtr(), true));
