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

Unified Diff: chrome/browser/printing/printer_query.h

Issue 668093002: Standardize usage of virtual/override/final in chrome/browser/printing/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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
Index: chrome/browser/printing/printer_query.h
diff --git a/chrome/browser/printing/printer_query.h b/chrome/browser/printing/printer_query.h
index 9fd9a82b33ea5f0ccfcb84642324392806479bd9..dcabfdd0d13cd1cf5073ac6c82a73207808c9924 100644
--- a/chrome/browser/printing/printer_query.h
+++ b/chrome/browser/printing/printer_query.h
@@ -32,11 +32,11 @@ class PrinterQuery : public PrintJobWorkerOwner {
PrinterQuery(int render_process_id, int render_view_id);
// PrintJobWorkerOwner implementation.
- virtual void GetSettingsDone(const PrintSettings& new_settings,
- PrintingContext::Result result) override;
- virtual PrintJobWorker* DetachWorker(PrintJobWorkerOwner* new_owner) override;
- virtual const PrintSettings& settings() const override;
- virtual int cookie() const override;
+ void GetSettingsDone(const PrintSettings& new_settings,
+ PrintingContext::Result result) override;
+ PrintJobWorker* DetachWorker(PrintJobWorkerOwner* new_owner) override;
+ const PrintSettings& settings() const override;
+ int cookie() const override;
// Initializes the printing context. It is fine to call this function multiple
// times to reinitialize the settings. |web_contents_observer| can be queried
@@ -65,7 +65,7 @@ class PrinterQuery : public PrintJobWorkerOwner {
bool is_valid() const;
private:
- virtual ~PrinterQuery();
+ ~PrinterQuery() override;
// Lazy create the worker thread. There is one worker thread per print job.
void StartWorker(const base::Closure& callback);
« no previous file with comments | « chrome/browser/printing/print_view_manager_base.h ('k') | chrome/browser/printing/printing_message_filter.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698