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

Unified Diff: chrome/browser/media_galleries/fileapi/itunes_file_util_unittest.cc

Issue 660343006: Standardize usage of virtual/override/final in chrome/browser/media_galleries (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/media_galleries/fileapi/itunes_file_util_unittest.cc
diff --git a/chrome/browser/media_galleries/fileapi/itunes_file_util_unittest.cc b/chrome/browser/media_galleries/fileapi/itunes_file_util_unittest.cc
index d6bb4749d178cb5432eee38b9e641a2d40d627f8..e2f301bb0d6c22c3116b51ebd3fa9176cb5df781 100644
--- a/chrome/browser/media_galleries/fileapi/itunes_file_util_unittest.cc
+++ b/chrome/browser/media_galleries/fileapi/itunes_file_util_unittest.cc
@@ -71,13 +71,13 @@ class TestITunesDataProvider : public ITunesDataProvider {
EXPECT_TRUE(fake_auto_add_dir_.CreateUniqueTempDir());
}
- virtual ~TestITunesDataProvider() {}
+ ~TestITunesDataProvider() override {}
- virtual void RefreshData(const ReadyCallback& ready_callback) override {
+ void RefreshData(const ReadyCallback& ready_callback) override {
ready_callback.Run(true /* success */);
}
- virtual const base::FilePath& auto_add_path() const override {
+ const base::FilePath& auto_add_path() const override {
return fake_auto_add_dir_.path();
}
@@ -101,12 +101,10 @@ class TestITunesFileUtil : public ITunesFileUtil {
: ITunesFileUtil(media_path_filter),
data_provider_(data_provider) {
}
- virtual ~TestITunesFileUtil() {}
+ ~TestITunesFileUtil() override {}
private:
- virtual ITunesDataProvider* GetDataProvider() override {
- return data_provider_;
- }
+ ITunesDataProvider* GetDataProvider() override { return data_provider_; }
ITunesDataProvider* data_provider_;
};
@@ -120,7 +118,7 @@ class TestMediaFileSystemBackend : public MediaFileSystemBackend {
MediaFileSystemBackend::MediaTaskRunner().get()),
test_file_util_(itunes_file_util) {}
- virtual storage::AsyncFileUtil* GetAsyncFileUtil(
+ storage::AsyncFileUtil* GetAsyncFileUtil(
storage::FileSystemType type) override {
if (type != storage::kFileSystemTypeItunes)
return NULL;

Powered by Google App Engine
This is Rietveld 408576698