diff options
author | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-19 23:17:07 +0000 |
---|---|---|
committer | jam@chromium.org <jam@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-10-19 23:17:07 +0000 |
commit | ad50def5e1165d0cc74b98f988bbd5962587d9f4 (patch) | |
tree | 3a751abaed2cc056ca60b8b1e3ff54532a6f4d3f /chrome/browser/extensions/crx_installer.cc | |
parent | 75c920505ddfb2d49c194c76bde64edd6b3f91f2 (diff) | |
download | chromium_src-ad50def5e1165d0cc74b98f988bbd5962587d9f4.zip chromium_src-ad50def5e1165d0cc74b98f988bbd5962587d9f4.tar.gz chromium_src-ad50def5e1165d0cc74b98f988bbd5962587d9f4.tar.bz2 |
Make NotificationService an interface in the content namespace, and switch callers to use it. Move the implementation to content/browser. Stop creating it in all child processes since it's only used in the browser.
BUG=98716
Review URL: http://codereview.chromium.org/8342048
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@106403 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/extensions/crx_installer.cc')
-rw-r--r-- | chrome/browser/extensions/crx_installer.cc | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/chrome/browser/extensions/crx_installer.cc b/chrome/browser/extensions/crx_installer.cc index 86ea255..614757f 100644 --- a/chrome/browser/extensions/crx_installer.cc +++ b/chrome/browser/extensions/crx_installer.cc @@ -34,7 +34,7 @@ #include "chrome/common/extensions/extension_file_util.h" #include "content/browser/browser_thread.h" #include "content/browser/user_metrics.h" -#include "content/common/notification_service.h" +#include "content/public/browser/notification_service.h" #include "grit/chromium_strings.h" #include "grit/generated_resources.h" #include "grit/theme_resources.h" @@ -444,10 +444,11 @@ void CrxInstaller::InstallUIAbort(bool user_initiated) { extension_, histogram_name.c_str()); // Kill the theme loading bubble. - NotificationService* service = NotificationService::current(); + content::NotificationService* service = + content::NotificationService::current(); service->Notify(chrome::NOTIFICATION_NO_THEME_DETECTED, content::Source<CrxInstaller>(this), - NotificationService::NoDetails()); + content::NotificationService::NoDetails()); Release(); // balanced in ConfirmInstall(). NotifyCrxInstallComplete(); @@ -518,7 +519,8 @@ void CrxInstaller::ReportFailureFromFileThread(const std::string& error) { void CrxInstaller::ReportFailureFromUIThread(const std::string& error) { DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); - NotificationService* service = NotificationService::current(); + content::NotificationService* service = + content::NotificationService::current(); service->Notify(chrome::NOTIFICATION_EXTENSION_INSTALL_ERROR, content::Source<CrxInstaller>(this), content::Details<const std::string>(&error)); @@ -594,8 +596,8 @@ void CrxInstaller::NotifyCrxInstallComplete() { // is problematic because they don't know anything about the // extension before it is unpacked, so they can not filter based // on the extension. - NotificationService::current()->Notify( + content::NotificationService::current()->Notify( chrome::NOTIFICATION_CRX_INSTALLER_DONE, content::Source<CrxInstaller>(this), - NotificationService::NoDetails()); + content::NotificationService::NoDetails()); } |