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

Unified Diff: components/offline_pages/offline_page_model_impl.cc

Issue 2342443006: [Offline pages] Use the new policy bits (Closed)
Patch Set: code review update 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 side-by-side diff with in-line comments
Download patch
Index: components/offline_pages/offline_page_model_impl.cc
diff --git a/components/offline_pages/offline_page_model_impl.cc b/components/offline_pages/offline_page_model_impl.cc
index cfb67fc33925267e49eabbd588ec40d6ad2eda6f..0414fb30be689c658dbaa90cb1057aceec63efdd 100644
--- a/components/offline_pages/offline_page_model_impl.cc
+++ b/components/offline_pages/offline_page_model_impl.cc
@@ -471,7 +471,7 @@ void OfflinePageModelImpl::DoDeleteCachedPagesByURLPredicate(
std::vector<int64_t> offline_ids;
for (const auto& id_page_pair : offline_pages_) {
- if (!IsUserRequestedPage(id_page_pair.second) &&
+ if (IsRemovedOnCacheReset(id_page_pair.second) &&
predicate.Run(id_page_pair.second.url))
offline_ids.push_back(id_page_pair.first);
}
@@ -1128,10 +1128,10 @@ void OfflinePageModelImpl::PostClearStorageIfNeededTask() {
weak_ptr_factory_.GetWeakPtr())));
}
-bool OfflinePageModelImpl::IsUserRequestedPage(
+bool OfflinePageModelImpl::IsRemovedOnCacheReset(
const OfflinePageItem& offline_page) const {
- return (offline_page.client_id.name_space == kAsyncNamespace ||
- offline_page.client_id.name_space == kDownloadNamespace);
+ return policy_controller_->IsRemovedOnCacheReset(
+ offline_page.client_id.name_space);
}
void OfflinePageModelImpl::RunWhenLoaded(const base::Closure& task) {

Powered by Google App Engine
This is Rietveld 408576698