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

Unified Diff: components/offline_pages/offline_page_storage_manager.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_storage_manager.cc
diff --git a/components/offline_pages/offline_page_storage_manager.cc b/components/offline_pages/offline_page_storage_manager.cc
new file mode 100644
index 0000000000000000000000000000000000000000..dcb82455e2fdb8d75c4057ac3f4a5e8a10ec3a38
--- /dev/null
+++ b/components/offline_pages/offline_page_storage_manager.cc
@@ -0,0 +1,76 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "components/offline_pages/offline_page_storage_manager.h"
+
+#include "base/bind.h"
+#include "components/offline_pages/client_policy_controller.h"
+#include "components/offline_pages/offline_page_client_policy.h"
+#include "components/offline_pages/offline_page_item.h"
+#include "components/offline_pages/offline_page_model.h"
+
+using DeletePageResult = offline_pages::OfflinePageModel::DeletePageResult;
+using MultipleOfflinePageItemResult =
+ offline_pages::OfflinePageModel::MultipleOfflinePageItemResult;
+
+namespace offline_pages {
+
+OfflinePageStorageManager::OfflinePageStorageManager(OfflinePageModel* model)
+ : model_(model),
+ policy_controller_(model->GetPolicyController()),
+ in_progress_(false),
+ weak_ptr_factory_(this) {
+}
+
+OfflinePageStorageManager::~OfflinePageStorageManager() {
+}
+
+void OfflinePageStorageManager::ClearPagesIfNeeded(
+ const ClearPageCallback& callback) {
+ if (!ShouldClearPages())
+ return;
+ in_progress_ = true;
+ model_->GetAllPages(base::Bind(&OfflinePageStorageManager::PurgePages,
+ weak_ptr_factory_.GetWeakPtr(), callback));
+}
+
+void OfflinePageStorageManager::PurgePages(
+ const ClearPageCallback& callback,
+ const MultipleOfflinePageItemResult& pages) {
fgorski 2016/05/06 05:57:24 DCHECK(in_progress_);
romax 2016/05/07 02:22:42 Done.
+ std::vector<int64_t> offline_ids;
+ GetExpiredPageIds(pages, offline_ids);
+ model_->DeletePagesByOfflineId(
+ offline_ids,
+ base::Bind(&OfflinePageStorageManager::OnExpiredPagesDeleted,
+ weak_ptr_factory_.GetWeakPtr(), callback, offline_ids.size()));
+}
+
+void OfflinePageStorageManager::GetExpiredPageIds(
+ const MultipleOfflinePageItemResult& pages,
+ std::vector<int64_t>& offline_ids) {
+ for (const auto& page : pages) {
+ if (IsPageExpired(page))
+ offline_ids.push_back(page.offline_id);
+ }
+}
+
+void OfflinePageStorageManager::OnExpiredPagesDeleted(
+ const ClearPageCallback& callback,
+ int pages_cleared,
+ DeletePageResult result) {
fgorski 2016/05/06 05:57:24 in_progress_ = false;
romax 2016/05/07 02:22:42 Done.
+ callback.Run(pages_cleared, result);
+}
+
+bool OfflinePageStorageManager::ShouldClearPages() {
+ return !in_progress_ && model_->is_loaded();
+}
+
+bool OfflinePageStorageManager::IsPageExpired(const OfflinePageItem& page) {
+ base::Time now = base::Time::Now();
+ LifetimePolicy policy =
fgorski 2016/05/06 05:57:24 const ref
romax 2016/05/07 02:22:42 Done.
+ policy_controller_->GetPolicy(page.client_id.name_space).lifetime_policy;
+ return now - page.last_access_time > policy.expiration_period;
+}
+
+} // namespace offline_pages

Powered by Google App Engine
This is Rietveld 408576698