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

Unified Diff: chrome/browser/printing/print_job_unittest.cc

Issue 624173002: replace OVERRIDE and FINAL with override and final in chrome/browser/[j-q]* (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase on master Created 6 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
« no previous file with comments | « chrome/browser/printing/print_job_manager.h ('k') | chrome/browser/printing/print_job_worker.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/printing/print_job_unittest.cc
diff --git a/chrome/browser/printing/print_job_unittest.cc b/chrome/browser/printing/print_job_unittest.cc
index 12a3dd9d50537acecef83929e638c457745e6074..118c4b188f688cc6e24a417b7b23e279cbc64080 100644
--- a/chrome/browser/printing/print_job_unittest.cc
+++ b/chrome/browser/printing/print_job_unittest.cc
@@ -19,7 +19,7 @@ namespace {
class TestSource : public printing::PrintedPagesSource {
public:
- virtual base::string16 RenderSourceName() OVERRIDE {
+ virtual base::string16 RenderSourceName() override {
return base::string16();
}
};
@@ -37,11 +37,11 @@ class TestOwner : public printing::PrintJobWorkerOwner {
public:
virtual void GetSettingsDone(
const printing::PrintSettings& new_settings,
- printing::PrintingContext::Result result) OVERRIDE {
+ printing::PrintingContext::Result result) override {
EXPECT_FALSE(true);
}
virtual printing::PrintJobWorker* DetachWorker(
- printing::PrintJobWorkerOwner* new_owner) OVERRIDE {
+ 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));
@@ -50,10 +50,10 @@ class TestOwner : public printing::PrintJobWorkerOwner {
settings_ = worker->printing_context()->settings();
return worker;
}
- virtual const printing::PrintSettings& settings() const OVERRIDE {
+ virtual const printing::PrintSettings& settings() const override {
return settings_;
}
- virtual int cookie() const OVERRIDE {
+ virtual int cookie() const override {
return 42;
}
@@ -79,7 +79,7 @@ class TestPrintNotifObserv : public content::NotificationObserver {
// content::NotificationObserver
virtual void Observe(int type,
const content::NotificationSource& source,
- const content::NotificationDetails& details) OVERRIDE {
+ const content::NotificationDetails& details) override {
ADD_FAILURE();
}
};
« no previous file with comments | « chrome/browser/printing/print_job_manager.h ('k') | chrome/browser/printing/print_job_worker.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698