Added last_access_time to DownloadItem and History DB

chrome/browser/android/download/download_manager_service.cc

Index: chrome/browser/android/download/download_manager_service.cc
diff --git a/chrome/browser/android/download/download_manager_service.cc b/chrome/browser/android/download/download_manager_service.cc
index 65ebf678ea575d091ad2cf251a1ac5bd7f25fbbb..4f4c0c5448efce2b3f7bf08f93f2f0a5d9e19e5c 100644
--- a/chrome/browser/android/download/download_manager_service.cc
+++ b/chrome/browser/android/download/download_manager_service.cc
@@ -239,6 +239,20 @@ void DownloadManagerService::CheckForExternallyRemovedDownloads(
   manager->CheckForHistoryFilesRemoval();
 }
 
+void DownloadManagerService::UpdateLastAccessTime(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj,
+    const JavaParamRef<jstring>& jdownload_guid) {
+  std::string download_guid = ConvertJavaStringToUTF8(env, jdownload_guid);
+  content::DownloadManager* manager = GetDownloadManager(false);

[David Trainor- moved to gerrit, 2017/02/22 05:56:03]

Can we assume always not off the record?

[shaktisahu, 2017/02/23 06:55:47]

Oops. I guess I have to pass the is_off_the_record from Java.

+  if (!manager)
+    return;
+
+  content::DownloadItem* item = manager->GetDownloadByGuid(download_guid);
+  if (item)
+    item->UpdateLastAccessTime();
+}
+
 void DownloadManagerService::CancelDownload(
     JNIEnv* env,
     jobject obj,