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

Unified Diff: chrome/browser/download/download_test_file_activity_observer.cc

Issue 665253002: Standardize usage of virtual/override/final in chrome/browser/download/ (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/download/download_test_file_activity_observer.cc
diff --git a/chrome/browser/download/download_test_file_activity_observer.cc b/chrome/browser/download/download_test_file_activity_observer.cc
index 110fefffabf7f06636bbe75e2cf3b207df3d1ab7..b7e704bcbcd167896badc7827c2033e5ec99dfa5 100644
--- a/chrome/browser/download/download_test_file_activity_observer.cc
+++ b/chrome/browser/download/download_test_file_activity_observer.cc
@@ -31,7 +31,7 @@ class DownloadTestFileActivityObserver::MockDownloadManagerDelegate
content::DownloadItem::kInvalidId + 1);
}
- virtual ~MockDownloadManagerDelegate() {}
+ ~MockDownloadManagerDelegate() override {}
void EnableFileChooser(bool enable) {
file_chooser_enabled_ = enable;
@@ -48,18 +48,17 @@ class DownloadTestFileActivityObserver::MockDownloadManagerDelegate
}
protected:
-
- virtual void PromptUserForDownloadPath(content::DownloadItem* item,
- const base::FilePath& suggested_path,
- const FileSelectedCallback&
- callback) override {
+ void PromptUserForDownloadPath(
+ content::DownloadItem* item,
+ const base::FilePath& suggested_path,
+ const FileSelectedCallback& callback) override {
file_chooser_displayed_ = true;
base::MessageLoop::current()->PostTask(
FROM_HERE, base::Bind(callback, (file_chooser_enabled_ ? suggested_path
: base::FilePath())));
}
- virtual void OpenDownload(content::DownloadItem* item) override {}
+ void OpenDownload(content::DownloadItem* item) override {}
private:
bool file_chooser_enabled_;
« no previous file with comments | « chrome/browser/download/download_target_determiner_unittest.cc ('k') | chrome/browser/download/download_ui_controller.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698