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

Unified Diff: components/offline_pages/offline_page_storage_manager_unittest.cc

Issue 1965633002: [Offline Pages] Introducing StorageManagerClient interface. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebasing after extracting callbacks. Created 4 years, 7 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: components/offline_pages/offline_page_storage_manager_unittest.cc
diff --git a/components/offline_pages/offline_page_storage_manager_unittest.cc b/components/offline_pages/offline_page_storage_manager_unittest.cc
index 426789531ee2015f106bcbb5aaea5ba137c470e7..7e130f966ff1eeeb6c959e93fead8b976a809e18 100644
--- a/components/offline_pages/offline_page_storage_manager_unittest.cc
+++ b/components/offline_pages/offline_page_storage_manager_unittest.cc
@@ -11,8 +11,8 @@
#include "base/time/time.h"
#include "components/offline_pages/client_policy_controller.h"
#include "components/offline_pages/offline_page_item.h"
-#include "components/offline_pages/offline_page_model.h"
#include "components/offline_pages/offline_page_types.h"
+#include "components/offline_pages/storage_manager_client.h"
#include "testing/gtest/include/gtest/gtest.h"
namespace offline_pages {
@@ -29,9 +29,10 @@ const int64_t kOfflineId = 1234LL;
const int64_t kOfflineId2 = 2345LL;
} // namespace
-class OfflinePageTestModel : public OfflinePageModel {
+class StorageManagerTestClient : public StorageManagerClient {
public:
- OfflinePageTestModel() : policy_controller_(new ClientPolicyController()) {
+ StorageManagerTestClient()
+ : policy_controller_(new ClientPolicyController()) {
// Manually adding pages for the fake model.
// Only the first page is expired.
pages_.clear();
@@ -57,7 +58,7 @@ class OfflinePageTestModel : public OfflinePageModel {
return policy_controller_.get();
}
- bool is_loaded() const override { return true; }
+ bool IsLoaded() const override { return true; }
private:
std::vector<OfflinePageItem> pages_;
@@ -82,7 +83,7 @@ class OfflinePageStorageManagerTest : public testing::Test {
private:
std::unique_ptr<OfflinePageStorageManager> manager_;
- std::unique_ptr<OfflinePageModel> model_;
+ std::unique_ptr<StorageManagerClient> client_;
int last_cleared_page_count_;
DeletePageResult last_delete_page_result_;
@@ -93,13 +94,13 @@ OfflinePageStorageManagerTest::OfflinePageStorageManagerTest()
last_delete_page_result_(DeletePageResult::SUCCESS) {}
void OfflinePageStorageManagerTest::SetUp() {
- model_.reset(new OfflinePageTestModel());
- manager_.reset(new OfflinePageStorageManager(model_.get()));
+ client_.reset(new StorageManagerTestClient());
+ manager_.reset(new OfflinePageStorageManager(client_.get()));
}
void OfflinePageStorageManagerTest::TearDown() {
manager_.reset();
- model_.reset();
+ client_.reset();
}
void OfflinePageStorageManagerTest::OnPagesCleared(int pages_cleared_count,

Powered by Google App Engine
This is Rietveld 408576698