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

Unified Diff: components/offline_pages/offline_page_metadata_store_impl.cc

Issue 1345043002: Update access info when an offline page is being visited (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Some more fixes Created 5 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_metadata_store_impl.cc
diff --git a/components/offline_pages/offline_page_metadata_store_impl.cc b/components/offline_pages/offline_page_metadata_store_impl.cc
index 90b7b45f556cf39967ec57ab84261f8a11ddf337..b9801f749158d355aeb2015333df8c67cd6d28f9 100644
--- a/components/offline_pages/offline_page_metadata_store_impl.cc
+++ b/components/offline_pages/offline_page_metadata_store_impl.cc
@@ -43,6 +43,7 @@ void OfflinePageItemToEntry(const OfflinePageItem& item,
item_proto->set_file_size(item.file_size);
item_proto->set_creation_time(item.creation_time.ToInternalValue());
item_proto->set_last_access_time(item.last_access_time.ToInternalValue());
+ item_proto->set_access_count(item.access_count);
}
bool OfflinePageItemFromEntry(const offline_pages::OfflinePageEntry& item_proto,
@@ -71,6 +72,9 @@ bool OfflinePageItemFromEntry(const offline_pages::OfflinePageEntry& item_proto,
item->last_access_time =
base::Time::FromInternalValue(item_proto.last_access_time());
}
+ if (item_proto.has_access_count()) {
+ item->access_count = item_proto.access_count();
+ }
return true;
}
@@ -155,7 +159,7 @@ void OfflinePageMetadataStoreImpl::Load(const LoadCallback& callback) {
weak_ptr_factory_.GetWeakPtr())));
}
-void OfflinePageMetadataStoreImpl::AddOfflinePage(
+void OfflinePageMetadataStoreImpl::AddOrUpdateOfflinePage(
const OfflinePageItem& offline_page_item,
const UpdateCallback& callback) {
scoped_ptr<ProtoDatabase<OfflinePageEntry>::KeyEntryVector> entries_to_save(

Powered by Google App Engine
This is Rietveld 408576698