diff options
author | rsleevi@chromium.org <rsleevi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-02-07 03:02:08 +0000 |
---|---|---|
committer | rsleevi@chromium.org <rsleevi@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-02-07 03:02:08 +0000 |
commit | b94584a231b160d2570cc38e78457fb852eb0c41 (patch) | |
tree | 8d03a214b0f7305c38eb8e2b6852015ae38fef07 /chrome/browser/printing/print_job_unittest.cc | |
parent | 8bc6596e0bdb39d1450d595bcc62244a5f3190b9 (diff) | |
download | chromium_src-b94584a231b160d2570cc38e78457fb852eb0c41.zip chromium_src-b94584a231b160d2570cc38e78457fb852eb0c41.tar.gz chromium_src-b94584a231b160d2570cc38e78457fb852eb0c41.tar.bz2 |
Linux/ChromeOS Chromium style checker cleanup, chrome/browser edition.
Automated clean up of style checker errors that were missed due to the plugin
not being executed on implementation files.
BUG=115047
Review URL: https://codereview.chromium.org/12212048
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@181164 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 | 17 |
1 files changed, 9 insertions, 8 deletions
diff --git a/chrome/browser/printing/print_job_unittest.cc b/chrome/browser/printing/print_job_unittest.cc index b9d5f47..f42a729 100644 --- a/chrome/browser/printing/print_job_unittest.cc +++ b/chrome/browser/printing/print_job_unittest.cc @@ -31,12 +31,13 @@ class TestPrintJobWorker : public printing::PrintJobWorker { class TestOwner : public printing::PrintJobWorkerOwner { public: - virtual void GetSettingsDone(const printing::PrintSettings& new_settings, - printing::PrintingContext::Result result) { + virtual void GetSettingsDone( + const printing::PrintSettings& new_settings, + printing::PrintingContext::Result result) OVERRIDE { EXPECT_FALSE(true); } virtual printing::PrintJobWorker* DetachWorker( - printing::PrintJobWorkerOwner* new_owner) { + printing::PrintJobWorkerOwner* new_owner) OVERRIDE { // We're screwing up here since we're calling worker from the main thread. // That's fine for testing. It is actually simulating PrinterQuery behavior. TestPrintJobWorker* worker(new TestPrintJobWorker(new_owner)); @@ -45,14 +46,14 @@ class TestOwner : public printing::PrintJobWorkerOwner { settings_ = worker->printing_context()->settings(); return worker; } - virtual MessageLoop* message_loop() { + virtual MessageLoop* message_loop() OVERRIDE { EXPECT_FALSE(true); return NULL; } - virtual const printing::PrintSettings& settings() const { + virtual const printing::PrintSettings& settings() const OVERRIDE { return settings_; } - virtual int cookie() const { + virtual int cookie() const OVERRIDE { return 42; } @@ -67,7 +68,7 @@ class TestPrintJob : public printing::PrintJob { explicit TestPrintJob(volatile bool* check) : check_(check) { } private: - ~TestPrintJob() { + virtual ~TestPrintJob() { *check_ = true; } volatile bool* check_; @@ -78,7 +79,7 @@ class TestPrintNotifObserv : public content::NotificationObserver { // content::NotificationObserver virtual void Observe(int type, const content::NotificationSource& source, - const content::NotificationDetails& details) { + const content::NotificationDetails& details) OVERRIDE { ADD_FAILURE(); } }; |