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

Unified Diff: components/offline_pages/offline_page_model.cc

Issue 1947323002: [Offline Pages] Implement OfflinePageStorageManager. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: More comments. 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_model.cc
diff --git a/components/offline_pages/offline_page_model.cc b/components/offline_pages/offline_page_model.cc
index d6bbddab2e76c6532e78ccda04101772ca69594d..6190486c409f20da8679bf33c084652ea352dc90 100644
--- a/components/offline_pages/offline_page_model.cc
+++ b/components/offline_pages/offline_page_model.cc
@@ -18,7 +18,9 @@
#include "base/strings/string_number_conversions.h"
#include "base/thread_task_runner_handle.h"
#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_storage_manager.h"
#include "components/offline_pages/proto/offline_pages.pb.h"
#include "url/gurl.h"
@@ -115,6 +117,8 @@ OfflinePageModel::OfflinePageModel(
archives_dir_(archives_dir),
is_loaded_(false),
task_runner_(task_runner),
+ policy_controller_(std::move(new ClientPolicyController())),
fgorski 2016/05/06 05:57:24 I don't think you need std::move here. unique poin
romax 2016/05/07 02:22:42 Done.
+ storage_manager_(std::move(new OfflinePageStorageManager(this))),
fgorski 2016/05/06 05:57:24 same
romax 2016/05/07 02:22:42 Done.
weak_ptr_factory_(this) {
task_runner_->PostTaskAndReply(
FROM_HERE, base::Bind(EnsureArchivesDirCreated, archives_dir_),
@@ -497,10 +501,18 @@ void OfflinePageModel::RecordStorageHistograms(int64_t total_space_bytes,
}
}
+ClientPolicyController* OfflinePageModel::GetPolicyController() {
+ return policy_controller_.get();
+}
+
OfflinePageMetadataStore* OfflinePageModel::GetStoreForTesting() {
return store_.get();
}
+OfflinePageStorageManager* OfflinePageModel::GetStorageManager() {
+ return storage_manager_.get();
+}
+
void OfflinePageModel::OnCreateArchiveDone(const GURL& requested_url,
int64_t offline_id,
const ClientId& client_id,

Powered by Google App Engine
This is Rietveld 408576698