diff options
author | phajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-08-03 23:02:21 +0000 |
---|---|---|
committer | phajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-08-03 23:02:21 +0000 |
commit | a5f9f373450f82d5a0a107473da6cd5f232d4ec9 (patch) | |
tree | 27ced074b40f2e39d664b1bd070de45718b4d4c8 /chrome/browser/printing/print_job_unittest.cc | |
parent | bf3fbd6347853955d7f3df8603e19b81db13a9f7 (diff) | |
download | chromium_src-a5f9f373450f82d5a0a107473da6cd5f232d4ec9.zip chromium_src-a5f9f373450f82d5a0a107473da6cd5f232d4ec9.tar.gz chromium_src-a5f9f373450f82d5a0a107473da6cd5f232d4ec9.tar.bz2 |
GTTF: Use a fresh TestingBrowserProcess for each test, part #4
For a detailed description see part #1, http://codereview.chromium.org/6478005/
BUG=61062
TEST=unit_tests, possibly more
Review URL: http://codereview.chromium.org/7465041
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@95350 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 | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/chrome/browser/printing/print_job_unittest.cc b/chrome/browser/printing/print_job_unittest.cc index 4ca676c..9e34875 100644 --- a/chrome/browser/printing/print_job_unittest.cc +++ b/chrome/browser/printing/print_job_unittest.cc @@ -7,6 +7,7 @@ #include "chrome/browser/printing/print_job.h" #include "chrome/browser/printing/print_job_worker.h" #include "chrome/common/chrome_notification_types.h" +#include "chrome/test/testing_browser_process_test.h" #include "content/common/notification_registrar.h" #include "content/common/notification_service.h" #include "googleurl/src/gurl.h" @@ -80,13 +81,15 @@ class TestPrintNotifObserv : public NotificationObserver { virtual void Observe(int type, const NotificationSource& source, const NotificationDetails& details) { - EXPECT_FALSE(true); + ADD_FAILURE(); } }; } // namespace -TEST(PrintJobTest, SimplePrint) { +typedef TestingBrowserProcessTest PrintJobTest; + +TEST_F(PrintJobTest, SimplePrint) { // Test the multi-threaded nature of PrintJob to make sure we can use it with // known lifetime. @@ -108,7 +111,7 @@ TEST(PrintJobTest, SimplePrint) { EXPECT_TRUE(check); } -TEST(PrintJobTest, SimplePrintLateInit) { +TEST_F(PrintJobTest, SimplePrintLateInit) { volatile bool check = false; MessageLoop current; scoped_refptr<printing::PrintJob> job(new TestPrintJob(&check)); |