diff options
| author | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-21 20:33:17 +0000 |
|---|---|---|
| committer | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-21 20:33:17 +0000 |
| commit | 0f08bf3692c1db8ff2032a7e944335cb5bb6a453 (patch) | |
| tree | 2348461db1b9ae039f92b810f6c44398532a6723 /chrome/browser/printing/print_job_unittest.cc | |
| parent | cbce8ffb3d90ed63e9fb36407d31f7f6ce8a991d (diff) | |
| download | chromium_src-0f08bf3692c1db8ff2032a7e944335cb5bb6a453.zip chromium_src-0f08bf3692c1db8ff2032a7e944335cb5bb6a453.tar.gz chromium_src-0f08bf3692c1db8ff2032a7e944335cb5bb6a453.tar.bz2 | |
Use a NotificationRegistrar to listen for notifications.
BUG=2381
Review URL: http://codereview.chromium.org/115647
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@16650 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/printing/print_job_unittest.cc')
| -rw-r--r-- | chrome/browser/printing/print_job_unittest.cc | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/chrome/browser/printing/print_job_unittest.cc b/chrome/browser/printing/print_job_unittest.cc index 4a8ce5f..e3eead4 100644 --- a/chrome/browser/printing/print_job_unittest.cc +++ b/chrome/browser/printing/print_job_unittest.cc @@ -6,6 +6,7 @@ #include "chrome/browser/printing/print_job.h" #include "chrome/browser/printing/print_job_worker.h" #include "chrome/browser/printing/printed_pages_source.h" +#include "chrome/common/notification_registrar.h" #include "chrome/common/notification_service.h" #include "googleurl/src/gurl.h" #include "testing/gtest/include/gtest/gtest.h" @@ -96,10 +97,10 @@ TEST(PrintJobTest, DISABLED_SimplePrint) { // This message loop is actually never run. MessageLoop current; + NotificationRegistrar registrar_; TestPrintNotifObserv observ; - NotificationService::current()->AddObserver( - &observ, NotificationType::ALL, - NotificationService::AllSources()); + registrar_.Add(&observ, NotificationType::ALL, + NotificationService::AllSources()); volatile bool check = false; scoped_refptr<printing::PrintJob> job(new TestPrintJob(&check)); EXPECT_EQ(MessageLoop::current(), job->message_loop()); @@ -109,9 +110,6 @@ TEST(PrintJobTest, DISABLED_SimplePrint) { job->Stop(); job = NULL; EXPECT_TRUE(check); - NotificationService::current()->RemoveObserver( - &observ, NotificationType::ALL, - NotificationService::AllSources()); } TEST(PrintJobTest, SimplePrintLateInit) { |