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

Unified Diff: chrome/browser/download/download_history.cc

Issue 2705283003: Added last_access_time to DownloadItem and History DB (Closed)
Patch Set: rebase origin/master Created 3 years, 10 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: chrome/browser/download/download_history.cc
diff --git a/chrome/browser/download/download_history.cc b/chrome/browser/download/download_history.cc
index c1f0721b15dca55ad85e206b3a31cf6b2c0e1e9a..e55b5d23891a8f43149a82395ee06cb8e6b383f6 100644
--- a/chrome/browser/download/download_history.cc
+++ b/chrome/browser/download/download_history.cc
@@ -141,7 +141,7 @@ history::DownloadRow GetDownloadRow(
history::ToHistoryDownloadInterruptReason(item->GetLastReason()),
std::string(), // Hash value (not available yet)
history::ToHistoryDownloadId(item->GetId()), item->GetGuid(),
- item->GetOpened(), by_ext_id, by_ext_name,
+ item->GetOpened(), item->GetLastAccessTime(), by_ext_id, by_ext_name,
history::GetHistoryDownloadSliceInfos(*item));
}
@@ -178,6 +178,7 @@ ShouldUpdateHistoryResult ShouldUpdateHistory(
(previous->interrupt_reason != current.interrupt_reason) ||
(previous->hash != current.hash) ||
(previous->opened != current.opened) ||
+ (previous->last_access_time != current.last_access_time) ||
(previous->by_ext_id != current.by_ext_id) ||
(previous->by_ext_name != current.by_ext_name) ||
(previous->download_slice_info != current.download_slice_info)) {
@@ -298,7 +299,8 @@ void DownloadHistory::QueryCallback(std::unique_ptr<InfoVector> infos) {
history::ToContentDownloadState(it->state),
history::ToContentDownloadDangerType(it->danger_type),
history::ToContentDownloadInterruptReason(it->interrupt_reason),
- it->opened, history::ToContentReceivedSlices(it->download_slice_info));
+ it->opened, it->last_access_time,
+ history::ToContentReceivedSlices(it->download_slice_info));
#if BUILDFLAG(ENABLE_EXTENSIONS)
if (!it->by_ext_id.empty() && !it->by_ext_name.empty()) {
new extensions::DownloadedByExtension(
« no previous file with comments | « chrome/browser/browsing_data/downloads_counter_browsertest.cc ('k') | chrome/browser/download/download_history_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698