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

Side by Side Diff: content/browser/fileapi/file_system_dir_url_request_job_unittest.cc

Issue 678073006: Standardize usage of virtual/override/final specifiers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix bogus formatting Created 6 years, 1 month 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "storage/browser/fileapi/file_system_dir_url_request_job.h" 5 #include "storage/browser/fileapi/file_system_dir_url_request_job.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/files/file_path.h" 9 #include "base/files/file_path.h"
10 #include "base/files/file_util.h" 10 #include "base/files/file_util.h"
(...skipping 94 matching lines...) Expand 10 before | Expand all | Expand 10 after
105 105
106 106
107 } // namespace 107 } // namespace
108 108
109 class FileSystemDirURLRequestJobTest : public testing::Test { 109 class FileSystemDirURLRequestJobTest : public testing::Test {
110 protected: 110 protected:
111 FileSystemDirURLRequestJobTest() 111 FileSystemDirURLRequestJobTest()
112 : weak_factory_(this) { 112 : weak_factory_(this) {
113 } 113 }
114 114
115 virtual void SetUp() override { 115 void SetUp() override {
116 ASSERT_TRUE(temp_dir_.CreateUniqueTempDir()); 116 ASSERT_TRUE(temp_dir_.CreateUniqueTempDir());
117 117
118 special_storage_policy_ = new MockSpecialStoragePolicy; 118 special_storage_policy_ = new MockSpecialStoragePolicy;
119 file_system_context_ = CreateFileSystemContextForTesting( 119 file_system_context_ = CreateFileSystemContextForTesting(
120 NULL, temp_dir_.path()); 120 NULL, temp_dir_.path());
121 121
122 file_system_context_->OpenFileSystem( 122 file_system_context_->OpenFileSystem(
123 GURL("http://remote/"), 123 GURL("http://remote/"),
124 storage::kFileSystemTypeTemporary, 124 storage::kFileSystemTypeTemporary,
125 storage::OPEN_FILE_SYSTEM_CREATE_IF_NONEXISTENT, 125 storage::OPEN_FILE_SYSTEM_CREATE_IF_NONEXISTENT,
126 base::Bind(&FileSystemDirURLRequestJobTest::OnOpenFileSystem, 126 base::Bind(&FileSystemDirURLRequestJobTest::OnOpenFileSystem,
127 weak_factory_.GetWeakPtr())); 127 weak_factory_.GetWeakPtr()));
128 base::RunLoop().RunUntilIdle(); 128 base::RunLoop().RunUntilIdle();
129 } 129 }
130 130
131 virtual void TearDown() override { 131 void TearDown() override {
132 // NOTE: order matters, request must die before delegate 132 // NOTE: order matters, request must die before delegate
133 request_.reset(NULL); 133 request_.reset(NULL);
134 delegate_.reset(NULL); 134 delegate_.reset(NULL);
135 } 135 }
136 136
137 void SetUpAutoMountContext(base::FilePath* mnt_point) { 137 void SetUpAutoMountContext(base::FilePath* mnt_point) {
138 *mnt_point = temp_dir_.path().AppendASCII("auto_mount_dir"); 138 *mnt_point = temp_dir_.path().AppendASCII("auto_mount_dir");
139 ASSERT_TRUE(base::CreateDirectory(*mnt_point)); 139 ASSERT_TRUE(base::CreateDirectory(*mnt_point));
140 140
141 ScopedVector<storage::FileSystemBackend> additional_providers; 141 ScopedVector<storage::FileSystemBackend> additional_providers;
(...skipping 290 matching lines...) Expand 10 before | Expand all | Expand 10 after
432 ASSERT_FALSE(request_->status().is_success()); 432 ASSERT_FALSE(request_->status().is_success());
433 EXPECT_EQ(net::ERR_FILE_NOT_FOUND, request_->status().error()); 433 EXPECT_EQ(net::ERR_FILE_NOT_FOUND, request_->status().error());
434 434
435 ASSERT_FALSE( 435 ASSERT_FALSE(
436 storage::ExternalMountPoints::GetSystemInstance()->RevokeFileSystem( 436 storage::ExternalMountPoints::GetSystemInstance()->RevokeFileSystem(
437 kValidExternalMountPoint)); 437 kValidExternalMountPoint));
438 } 438 }
439 439
440 } // namespace (anonymous) 440 } // namespace (anonymous)
441 } // namespace content 441 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698