Added last_access_time to DownloadItem and History DB

components/history/core/browser/download_row.h

Index: components/history/core/browser/download_row.h
diff --git a/components/history/core/browser/download_row.h b/components/history/core/browser/download_row.h
index 1c47635aeed0b0a8e56192d12f09b8d8ae043a0d..9c0539748757b1e361ae451e8509639de0623a4f 100644
--- a/components/history/core/browser/download_row.h
+++ b/components/history/core/browser/download_row.h
@@ -46,6 +46,7 @@ struct DownloadRow {
               DownloadId id,
               const std::string& guid,
               bool download_opened,
+              const base::Time& last_access,

[sdefresne, 2017/02/28 18:09:07]

base/time.h has a comment saying that all time classes should be passed by
value, can you change here and everywhere else?

// All time classes are copyable, assignable, and occupy 64-bits per instance.
// As a result, prefer passing them by value:
//   void MyFunction(TimeDelta arg);
// If circumstances require, you may also pass by const reference:
//   void MyFunction(const TimeDelta& arg);  // Not preferred.

https://cs.chromium.org/chromium/src/base/time/time.h?q=base::Time&sq=package...

[shaktisahu, 2017/02/28 21:07:54]

Done. Thanks!

               const std::string& ext_id,
               const std::string& ext_name,
               const std::vector<DownloadSliceInfo>& download_slice_info);
@@ -134,6 +135,9 @@ struct DownloadRow {
   // Whether this download has ever been opened from the browser.
   bool opened;
 
+  // The time when the download was last accessed.
+  base::Time last_access_time;
+
   // The id and name of the extension that created this download.
   std::string by_ext_id;
   std::string by_ext_name;