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

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

Issue 625113002: replace OVERRIDE and FINAL with override and final in chrome/browser/[a-i]* (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix newly added OVERRIDEs 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/chrome_download_manager_delegate_unittest.cc
diff --git a/chrome/browser/download/chrome_download_manager_delegate_unittest.cc b/chrome/browser/download/chrome_download_manager_delegate_unittest.cc
index bb67541f913ab12758de1ed78cc78d9cb94a33d2..bb4ffe4cbc62da47e529a30b0d42a566d28a39c9 100644
--- a/chrome/browser/download/chrome_download_manager_delegate_unittest.cc
+++ b/chrome/browser/download/chrome_download_manager_delegate_unittest.cc
@@ -77,14 +77,14 @@ class TestChromeDownloadManagerDelegate : public ChromeDownloadManagerDelegate {
virtual ~TestChromeDownloadManagerDelegate() {}
virtual safe_browsing::DownloadProtectionService*
- GetDownloadProtectionService() OVERRIDE {
+ GetDownloadProtectionService() override {
return NULL;
}
virtual void NotifyExtensions(
content::DownloadItem* download,
const base::FilePath& suggested_virtual_path,
- const NotifyExtensionsCallback& callback) OVERRIDE {
+ const NotifyExtensionsCallback& callback) override {
callback.Run(base::FilePath(),
DownloadPathReservationTracker::UNIQUIFY);
}
@@ -95,7 +95,7 @@ class TestChromeDownloadManagerDelegate : public ChromeDownloadManagerDelegate {
bool create_directory,
DownloadPathReservationTracker::FilenameConflictAction conflict_action,
const DownloadPathReservationTracker::ReservedPathCallback& callback)
- OVERRIDE {
+ override {
// Pretend the path reservation succeeded without any change to
// |target_path|.
base::MessageLoop::current()->PostTask(
@@ -106,7 +106,7 @@ class TestChromeDownloadManagerDelegate : public ChromeDownloadManagerDelegate {
DownloadItem* download,
const base::FilePath& suggested_path,
const DownloadTargetDeterminerDelegate::FileSelectedCallback& callback)
- OVERRIDE {
+ override {
base::FilePath return_path = MockPromptUserForDownloadPath(download,
suggested_path,
callback);
@@ -127,8 +127,8 @@ class ChromeDownloadManagerDelegateTest
ChromeDownloadManagerDelegateTest();
// ::testing::Test
- virtual void SetUp() OVERRIDE;
- virtual void TearDown() OVERRIDE;
+ virtual void SetUp() override;
+ virtual void TearDown() override;
// Verifies and clears test expectations for |delegate_| and
// |download_manager_|.
« no previous file with comments | « chrome/browser/download/chrome_download_manager_delegate.h ('k') | chrome/browser/download/download_browsertest.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698