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..911e3fc894f4a38cb640c880d8a3bd9d6663d85d 100644
--- a/chrome/browser/android/download/download_manager_service.cc
+++ b/chrome/browser/android/download/download_manager_service.cc
@@ -239,6 +239,22 @@ void DownloadManagerService::CheckForExternallyRemovedDownloads(
   manager->CheckForHistoryFilesRemoval();
 }
 
+void DownloadManagerService::UpdateLastAccessTime(
+    JNIEnv* env,
+    const JavaParamRef<jobject>& obj,
+    const JavaParamRef<jstring>& jdownload_guid,
+    jlong last_access_time,

[David Trainor- moved to gerrit, 2017/02/23 21:21:20]

We might not need this from Java, but I like that the native method takes the
time.  I'm fine either way though.

[shaktisahu, 2017/02/23 23:18:07]

Done. Removed from Java.

+    bool is_off_the_record) {
+  std::string download_guid = ConvertJavaStringToUTF8(env, jdownload_guid);
+  content::DownloadManager* manager = GetDownloadManager(is_off_the_record);
+  if (!manager)
+    return;
+
+  content::DownloadItem* item = manager->GetDownloadByGuid(download_guid);
+  if (item)
+    item->SetLastAccessTime(base::Time::FromJavaTime(last_access_time));
+}
+
 void DownloadManagerService::CancelDownload(
     JNIEnv* env,
     jobject obj,