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

Side by Side Diff: components/offline_pages/archive_manager_unittest.cc

Issue 2317123002: c/browser, c/common, components O-P: Change ScopedTempDir::path() to GetPath() (Closed)
Patch Set: Just rebased Created 4 years, 3 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 "components/offline_pages/archive_manager.h" 5 #include "components/offline_pages/archive_manager.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <memory> 8 #include <memory>
9 #include <set> 9 #include <set>
10 #include <vector> 10 #include <vector>
(...skipping 27 matching lines...) Expand all
38 void PumpLoop(); 38 void PumpLoop();
39 void ResetResults(); 39 void ResetResults();
40 40
41 void ResetManager(const base::FilePath& file_path); 41 void ResetManager(const base::FilePath& file_path);
42 void Callback(bool result); 42 void Callback(bool result);
43 void GetAllArchivesCallback(const std::set<base::FilePath>& archive_paths); 43 void GetAllArchivesCallback(const std::set<base::FilePath>& archive_paths);
44 void GetStorageStatsCallback( 44 void GetStorageStatsCallback(
45 const ArchiveManager::StorageStats& storage_sizes); 45 const ArchiveManager::StorageStats& storage_sizes);
46 46
47 ArchiveManager* manager() { return manager_.get(); } 47 ArchiveManager* manager() { return manager_.get(); }
48 const base::FilePath& temp_path() const { return temp_dir_.path(); } 48 const base::FilePath& temp_path() const { return temp_dir_.GetPath(); }
49 CallbackStatus callback_status() const { return callback_status_; } 49 CallbackStatus callback_status() const { return callback_status_; }
50 const std::set<base::FilePath>& last_archive_paths() const { 50 const std::set<base::FilePath>& last_archive_paths() const {
51 return last_archvie_paths_; 51 return last_archvie_paths_;
52 } 52 }
53 ArchiveManager::StorageStats last_storage_sizes() const { 53 ArchiveManager::StorageStats last_storage_sizes() const {
54 return last_storage_sizes_; 54 return last_storage_sizes_;
55 } 55 }
56 56
57 private: 57 private:
58 scoped_refptr<base::TestSimpleTaskRunner> task_runner_; 58 scoped_refptr<base::TestSimpleTaskRunner> task_runner_;
59 base::ThreadTaskRunnerHandle task_runner_handle_; 59 base::ThreadTaskRunnerHandle task_runner_handle_;
60 base::ScopedTempDir temp_dir_; 60 base::ScopedTempDir temp_dir_;
61 61
62 std::unique_ptr<ArchiveManager> manager_; 62 std::unique_ptr<ArchiveManager> manager_;
63 CallbackStatus callback_status_; 63 CallbackStatus callback_status_;
64 std::set<base::FilePath> last_archvie_paths_; 64 std::set<base::FilePath> last_archvie_paths_;
65 ArchiveManager::StorageStats last_storage_sizes_; 65 ArchiveManager::StorageStats last_storage_sizes_;
66 }; 66 };
67 67
68 ArchiveManagerTest::ArchiveManagerTest() 68 ArchiveManagerTest::ArchiveManagerTest()
69 : task_runner_(new base::TestSimpleTaskRunner), 69 : task_runner_(new base::TestSimpleTaskRunner),
70 task_runner_handle_(task_runner_), 70 task_runner_handle_(task_runner_),
71 callback_status_(CallbackStatus::NOT_CALLED), 71 callback_status_(CallbackStatus::NOT_CALLED),
72 last_storage_sizes_({0, 0}) {} 72 last_storage_sizes_({0, 0}) {}
73 73
74 void ArchiveManagerTest::SetUp() { 74 void ArchiveManagerTest::SetUp() {
75 ASSERT_TRUE(temp_dir_.CreateUniqueTempDir()); 75 ASSERT_TRUE(temp_dir_.CreateUniqueTempDir());
76 ResetManager(temp_dir_.path()); 76 ResetManager(temp_dir_.GetPath());
77 } 77 }
78 78
79 void ArchiveManagerTest::PumpLoop() { 79 void ArchiveManagerTest::PumpLoop() {
80 task_runner_->RunUntilIdle(); 80 task_runner_->RunUntilIdle();
81 } 81 }
82 82
83 void ArchiveManagerTest::ResetResults() { 83 void ArchiveManagerTest::ResetResults() {
84 callback_status_ = CallbackStatus::NOT_CALLED; 84 callback_status_ = CallbackStatus::NOT_CALLED;
85 last_archvie_paths_.clear(); 85 last_archvie_paths_.clear();
86 } 86 }
(...skipping 178 matching lines...) Expand 10 before | Expand all | Expand 10 after
265 265
266 manager()->GetStorageStats(base::Bind( 266 manager()->GetStorageStats(base::Bind(
267 &ArchiveManagerTest::GetStorageStatsCallback, base::Unretained(this))); 267 &ArchiveManagerTest::GetStorageStatsCallback, base::Unretained(this)));
268 PumpLoop(); 268 PumpLoop();
269 EXPECT_GT(last_storage_sizes().free_disk_space, 0); 269 EXPECT_GT(last_storage_sizes().free_disk_space, 0);
270 EXPECT_EQ(last_storage_sizes().total_archives_size, 270 EXPECT_EQ(last_storage_sizes().total_archives_size,
271 base::ComputeDirectorySize(temp_path())); 271 base::ComputeDirectorySize(temp_path()));
272 } 272 }
273 273
274 } // namespace offline_pages 274 } // namespace offline_pages
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698