Added last_access_time to DownloadItem and History DB

chrome/browser/android/download/download_manager_service.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/android/download/download_manager_service.h"
 
 #include "base/android/jni_string.h"
 #include "base/location.h"
 #include "base/single_thread_task_runner.h"
 #include "base/strings/string_number_conversions.h"
@@ skipping 221 matching lines @@
   // requests to check for externally removed downloads.
   if (!is_history_query_complete_)
     return;
 
   content::DownloadManager* manager = GetDownloadManager(is_off_the_record);
   if (!manager)
     return;
   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,
     const JavaParamRef<jstring>& jdownload_guid,
     bool is_off_the_record,
     bool is_notification_dismissed) {
   std::string download_guid = ConvertJavaStringToUTF8(env, jdownload_guid);
   DownloadController::RecordDownloadCancelReason(
       is_notification_dismissed ?
           DownloadController::CANCEL_REASON_NOTIFICATION_DISMISSED :
@@ skipping 209 matching lines @@
       chrome::android::kDownloadAutoResumptionThrottling,
       kAutoResumptionLimitVariation);
   int auto_resumption_limit;
   if (!variation.empty() &&
       base::StringToInt(variation, &auto_resumption_limit)) {
     return auto_resumption_limit;
   }
 
   return kDefaultAutoResumptionLimit;
 }