Added last_access_time to DownloadItem and History DB

content/public/browser/download_item.h

Index: content/public/browser/download_item.h
diff --git a/content/public/browser/download_item.h b/content/public/browser/download_item.h
index 32963bfd476415a5b22a75857900d68b462c7527..44d773df88d255b82bc04ce377aa811e4fe029ef 100644
--- a/content/public/browser/download_item.h
+++ b/content/public/browser/download_item.h
@@ -377,6 +377,10 @@ class CONTENT_EXPORT DownloadItem : public base::SupportsUserData {
   // Returns true if the download has been opened.
   virtual bool GetOpened() const = 0;
 
+  // Time the download was last accessed. Returns NULL if the download has never
+  // been opened.
+  virtual base::Time GetLastAccessTime() const = 0;
+
   //    Misc State accessors ---------------------------------------------------
 
   // BrowserContext that indirectly owns this download. Always valid.
@@ -403,6 +407,9 @@ class CONTENT_EXPORT DownloadItem : public base::SupportsUserData {
   // Mark the download as having been opened (without actually opening it).
   virtual void SetOpened(bool opened) = 0;
 
+  // Updates the last access time of the download.
+  virtual void UpdateLastAccessTime() = 0;

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

Should this just be SetLastAccessTime(Time)?  That would allow us to eventually
read the file data from disk to see the access time for externally opened files.

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

Good idea. Done.

+
   // Set a display name for the download that will be independent of the target
   // filename. If |name| is not empty, then GetFileNameToReportUser() will
   // return |name|. Has no effect on the final target filename.