Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1036)

Unified Diff: chrome/browser/sync/util/extensions_activity_monitor_unittest.cc

Issue 8342048: Make NotificationService an interface in the content namespace, and switch callers to use it. Mov... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/sync/util/extensions_activity_monitor_unittest.cc
===================================================================
--- chrome/browser/sync/util/extensions_activity_monitor_unittest.cc (revision 106380)
+++ chrome/browser/sync/util/extensions_activity_monitor_unittest.cc (working copy)
@@ -13,7 +13,7 @@
#include "chrome/common/extensions/extension.h"
#include "chrome/common/extensions/extension_constants.h"
#include "content/browser/browser_thread.h"
-#include "content/common/notification_service.h"
+#include "content/browser/notification_service_impl.h"
#include "testing/gtest/include/gtest/gtest.h"
using browser_sync::ExtensionsActivityMonitor;
@@ -46,7 +46,7 @@
extension_(e), function_(t), repeats_(repeats), done_(done) {}
virtual void Run() {
for (size_t i = 0; i < repeats_; i++) {
- NotificationService::current()->Notify(
+ content::NotificationService::current()->Notify(
chrome::NOTIFICATION_EXTENSION_BOOKMARKS_API_INVOKED,
content::Source<Extension>(extension_.get()),
content::Details<const BookmarksFunction>(function_.get()));
@@ -91,16 +91,16 @@
class DoUIThreadSetupTask : public Task {
public:
- DoUIThreadSetupTask(NotificationService** service,
+ DoUIThreadSetupTask(content::NotificationService** service,
base::WaitableEvent* done)
: service_(service), signal_when_done_(done) {}
virtual ~DoUIThreadSetupTask() {}
virtual void Run() {
- *service_ = new NotificationService();
+ *service_ = new NotificationServiceImpl();
signal_when_done_->Signal();
}
private:
- NotificationService** service_;
+ content::NotificationService** service_;
base::WaitableEvent* signal_when_done_;
DISALLOW_COPY_AND_ASSIGN(DoUIThreadSetupTask);
};
@@ -139,7 +139,7 @@
return extension->id();
}
private:
- NotificationService* service_;
+ content::NotificationService* service_;
BrowserThread ui_thread_;
};
« no previous file with comments | « chrome/browser/sync/util/extensions_activity_monitor.cc ('k') | chrome/browser/tab_contents/background_contents.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698