Added last_access_time to DownloadItem and History DB

content/public/browser/download_item.h

 // Copyright (c) 2012 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.
 //
 // Each download is represented by a DownloadItem, and all DownloadItems
 // are owned by the DownloadManager which maintains a global list of all
 // downloads. DownloadItems are created when a user initiates a download,
 // and exist for the duration of the browser life time.
 //
 // Download observers:
@@ skipping 359 matching lines @@
 
   // Returns true if the download will be auto-opened when complete.
   virtual bool GetOpenWhenComplete() const = 0;
 
   // Returns true if the download has been auto-opened by the system.
   virtual bool GetAutoOpened() = 0;
 
   // 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.
   virtual BrowserContext* GetBrowserContext() const = 0;
 
   // WebContents associated with the download. Returns nullptr if the
   // WebContents is unknown or if the download was not performed on behalf of a
   // renderer.
   virtual WebContents* GetWebContents() const = 0;
 
   // External state transitions/setters ----------------------------------------
 
   // TODO(rdsmith): These should all be removed; the download item should
   // control its own state transitions.
 
   // Called if a check of the download contents was performed and the results of
   // the test are available. This should only be called after AllDataSaved() is
   // true.
   virtual void OnContentCheckCompleted(DownloadDangerType danger_type) = 0;
 
   // Mark the download to be auto-opened when completed.
   virtual void SetOpenWhenComplete(bool open) = 0;
 
   // 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.
   virtual void SetDisplayName(const base::FilePath& name) = 0;
 
   // Debug/testing -------------------------------------------------------------
   virtual std::string DebugString(bool verbose) const = 0;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_DOWNLOAD_ITEM_H_