diff options
author | jamescook@chromium.org <jamescook@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-07-31 12:44:06 +0000 |
---|---|---|
committer | jamescook@chromium.org <jamescook@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-07-31 12:44:06 +0000 |
commit | adf5a10eeb5dc230f27d3b1344f2111bdfcd97d6 (patch) | |
tree | 8ba5fbfcb3ddaa54355fa3b03cbdcadcbc6d875d /chrome/browser/extensions/install_tracker.cc | |
parent | 248ee9be3db637800a1bf7b13817df43f904248b (diff) | |
download | chromium_src-adf5a10eeb5dc230f27d3b1344f2111bdfcd97d6.zip chromium_src-adf5a10eeb5dc230f27d3b1344f2111bdfcd97d6.tar.gz chromium_src-adf5a10eeb5dc230f27d3b1344f2111bdfcd97d6.tar.bz2 |
Move extension notifications to extensions/browser/notification_types.h
As part of the extensions module refactor we temporarily allowed the use
of chrome_notification_types.h in src/extensions. However, this allowed
code moving into src/extensions to listen to notifications from Chrome,
which was a layering violation.
This CL moves all the extensions notifications into their own header and
cleans up DEPS to disallow the exception.
BUG=392622
TEST=compiles, unit_tests, browser_tests, extensions_unittests
TBR=miket@chromium.org for apps/
Review URL: https://codereview.chromium.org/425303002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@286770 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/extensions/install_tracker.cc')
-rw-r--r-- | chrome/browser/extensions/install_tracker.cc | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/chrome/browser/extensions/install_tracker.cc b/chrome/browser/extensions/install_tracker.cc index ffa52fa..b3a272f 100644 --- a/chrome/browser/extensions/install_tracker.cc +++ b/chrome/browser/extensions/install_tracker.cc @@ -18,7 +18,7 @@ InstallTracker::InstallTracker(content::BrowserContext* browser_context, extensions::ExtensionPrefs* prefs) : extension_registry_observer_(this) { registrar_.Add(this, - chrome::NOTIFICATION_EXTENSION_UPDATE_DISABLED, + extensions::NOTIFICATION_EXTENSION_UPDATE_DISABLED, content::Source<content::BrowserContext>(browser_context)); registrar_.Add(this, chrome::NOTIFICATION_APP_INSTALLED_TO_APPLIST, @@ -29,7 +29,7 @@ InstallTracker::InstallTracker(content::BrowserContext* browser_context, if (prefs) { AppSorting* sorting = prefs->app_sorting(); registrar_.Add(this, - chrome::NOTIFICATION_EXTENSION_LAUNCHER_REORDERED, + chrome::NOTIFICATION_APP_LAUNCHER_REORDERED, content::Source<AppSorting>(sorting)); pref_change_registrar_.Init(prefs->pref_service()); pref_change_registrar_.Add( @@ -135,14 +135,14 @@ void InstallTracker::Observe(int type, const content::NotificationSource& source, const content::NotificationDetails& details) { switch (type) { - case chrome::NOTIFICATION_EXTENSION_UPDATE_DISABLED: { + case extensions::NOTIFICATION_EXTENSION_UPDATE_DISABLED: { const Extension* extension = content::Details<const Extension>(details).ptr(); FOR_EACH_OBSERVER( InstallObserver, observers_, OnDisabledExtensionUpdated(extension)); break; } - case chrome::NOTIFICATION_EXTENSION_LAUNCHER_REORDERED: { + case chrome::NOTIFICATION_APP_LAUNCHER_REORDERED: { FOR_EACH_OBSERVER(InstallObserver, observers_, OnAppsReordered()); break; } |