[downloads] Document undocumented accessors in DownloadItem.

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 64 matching lines @@
   enum TargetDisposition {
     TARGET_DISPOSITION_OVERWRITE, // Overwrite if the target already exists.
     TARGET_DISPOSITION_PROMPT     // Prompt the user for the actual
                                   // target. Implies
                                   // TARGET_DISPOSITION_OVERWRITE.
   };
 
   // Callback used with AcquireFileAndDeleteDownload().
   typedef base::Callback<void(const base::FilePath&)> AcquireFileCallback;
 
+  // Used to represent an invalid download ID.
   static const uint32_t kInvalidId;
 
   // Interface that observers of a particular download must implement in order
   // to receive updates to the download's status.
   class CONTENT_EXPORT Observer {
    public:
     virtual void OnDownloadUpdated(DownloadItem* download) {}
     virtual void OnDownloadOpened(DownloadItem* download) {}
     virtual void OnDownloadRemoved(DownloadItem* download) {}
 
@@ skipping 24 matching lines @@
   // path to the downloaded file. The caller is responsible for cleanup.
   // Note: It is important for |callback| to be valid since the downloaded file
   // will not be cleaned up if the callback fails.
   virtual void StealDangerousDownload(const AcquireFileCallback& callback) = 0;
 
   // Pause a download.  Will have no effect if the download is already
   // paused.
   virtual void Pause() = 0;
 
   // Resume a download that has been paused or interrupted. Will have no effect
-  // if the download is neither.
+  // if the download is neither. Only does something if CanResume() returns
+  // true.
   virtual void Resume() = 0;
 
-  // Cancel the download operation. We need to distinguish between cancels at
-  // exit (DownloadManager destructor) from user interface initiated cancels
-  // because at exit, the history system may not exist, and any updates to it
-  // require AddRef'ing the DownloadManager in the destructor which results in
-  // a DCHECK failure. Set |user_cancel| to false when canceling from at
-  // exit to prevent this crash. This may result in a difference between the
-  // downloaded file's size on disk, and what the history system's last record
-  // of it is. At worst, we'll end up re-downloading a small portion of the file
-  // when resuming a download (assuming the server supports byte ranges).
+  // Cancel the download operation.
+  //
+  // Set |user_cancel| to true if the cancellation was triggered by an explicit
+  // user action. Non-user-initiated cancels typically happen when the browser
+  // is being closed with in-progress downloads.
   virtual void Cancel(bool user_cancel) = 0;
 
   // Removes the download from the views and history. If the download was
   // in-progress or interrupted, then the intermediate file will also be
   // deleted.
   virtual void Remove() = 0;
 
   // Open the file associated with this download.  If the download is
   // still in progress, marks the download to be opened when it is complete.
   virtual void OpenDownload() = 0;
 
   // Show the download via the OS shell.
   virtual void ShowDownloadInShell() = 0;
 
   // State accessors -----------------------------------------------------------
 
   // Retrieve the ID for this download. The ID is provided by the owner of the
   // DownloadItem and is expected to uniquely identify the download within the
-  // context of its container during the lifetime of the download.
+  // context of its container during the lifetime of the download. A valid
+  // download will never return |kInvalidId|.
   virtual uint32_t GetId() const = 0;
 
   // Retrieve the GUID for this download. The returned string is never empty and
-  // will satisfy base::IsValidGUID(), in addition to uniquely identifying the
-  // download during its lifetime regardless of its container.
+  // will satisfy base::IsValidGUID(), and uniquely identifies the download

[clamy, 2016/10/04 14:21:30]

nit: remove comma?

[asanka, 2016/10/04 14:44:55]

Done

+  // during its lifetime.
   virtual const std::string& GetGuid() const = 0;
 
   // Get the current state of the download. See DownloadState for descriptions
   // of each download state.
   virtual DownloadState GetState() const = 0;
 
   // Returns the most recent interrupt reason for this download. Returns
-  // DOWNLOAD_INTERRUPT_REASON_NONE if there is no previous interrupt reason.
-  // Cancelled downloads return DOWNLOAD_INTERRUPT_REASON_USER_CANCELLED. If
-  // the download was resumed, then the return value is the interrupt reason
-  // prior to resumption.
+  // |DOWNLOAD_INTERRUPT_REASON_NONE| if there is no previous interrupt reason.
+  // Interrupted downloads and resumed downloads return the last known interrupt
+  // reason.
   virtual DownloadInterruptReason GetLastReason() const = 0;
 
+  // The download is currently paused. Calling Resume() will transition out of
+  // this paused state.
   virtual bool IsPaused() const = 0;
+
+  // DEPRECATED. True if this is a temporary download and should not be
+  // persisted.
   virtual bool IsTemporary() const = 0;
 
   // Returns true if the download can be resumed. A download can be resumed if
   // an in-progress download was paused or if an interrupted download requires
   // user-interaction to resume.
   virtual bool CanResume() const = 0;
 
   // Returns true if the download is in a terminal state. This includes
   // completed downloads, cancelled downloads, and interrupted downloads that
   // can't be resumed.
   virtual bool IsDone() const = 0;
 
   //    Origin State accessors -------------------------------------------------
 
+  // Final URL. The primary resource being downloaded is from this URL. This is
+  // the tail of GetUrlChain(). May return an empty GURL if there is no valid
+  // download URL.
   virtual const GURL& GetURL() const = 0;
+
+  // The complete URL chain including redirects. URL at index i redirected to
+  // URL at index i+1.
   virtual const std::vector<GURL>& GetUrlChain() const = 0;
+
+  // The URL that the download request originally attempted to fetch. This may
+  // defer from GetURL() if there were redirects. The return value from this

[clamy, 2016/10/04 14:21:29]

nit: s/defer/differ

[asanka, 2016/10/04 14:44:55]

Done

+  // accessor is the same as the head of GetUrlChain().
   virtual const GURL& GetOriginalUrl() const = 0;
+
+  // URL of document that is considered the referrer for the original URL.
   virtual const GURL& GetReferrerUrl() const = 0;
+
+  // Site instance URL. Used to locate the correct storage partition during
+  // subsequent browser sessions.

[clamy, 2016/10/04 14:21:30]

nit: mention that it may be different from the current & original URL?

[asanka, 2016/10/04 14:44:55]

Done

   virtual const GURL& GetSiteUrl() const = 0;
+
+  // URL of the top level frame at the time the download was initiated.
   virtual const GURL& GetTabUrl() const = 0;
+
+  // Referrer URL for top level frame.
   virtual const GURL& GetTabReferrerUrl() const = 0;
+
+  // For downloads initiated via <a download>, this is the suggested download
+  // filename from the download attribute.
   virtual std::string GetSuggestedFilename() const = 0;
+
+  // Content-Disposition header value from HTTP response.
   virtual std::string GetContentDisposition() const = 0;
+
+  // Effective MIME type of downloaded content.
   virtual std::string GetMimeType() const = 0;
+
+  // Content-Type header value from HTTP response. May be different from
+  // GetMimeType() if a different effective MIME type was chosen after MIME
+  // sniffing.
   virtual std::string GetOriginalMimeType() const = 0;
+
+  // Remote address of server serving download contents.
   virtual std::string GetRemoteAddress() const = 0;
+
+  // Whether the download request was initiated in response to a user gesture.
   virtual bool HasUserGesture() const = 0;
+
+  // The page transition type associated with the download request.
   virtual ui::PageTransition GetTransitionType() const = 0;
+
+  // Last-Modified header value.
   virtual const std::string& GetLastModifiedTime() const = 0;
+
+  // ETag header value.
   virtual const std::string& GetETag() const = 0;
+
+  // Whether this download is a SavePackage download.
   virtual bool IsSavePackageDownload() const = 0;
 
   //    Destination State accessors --------------------------------------------
 
   // Full path to the downloaded or downloading file. This is the path to the
   // physical file, if one exists. It should be considered a hint; changes to
   // this value and renames of the file on disk are not atomic with each other.
   // May be empty if the in-progress path hasn't been determined yet or if the
   // download was interrupted.
   //
@@ skipping 12 matching lines @@
 
   // If the download forced a path rather than requesting name determination,
   // return the path requested.
   virtual const base::FilePath& GetForcedFilePath() const = 0;
 
   // Returns the file-name that should be reported to the user. If a display
   // name has been explicitly set using SetDisplayName(), this function returns
   // that display name. Otherwise returns the final target filename.
   virtual base::FilePath GetFileNameToReportUser() const = 0;
 
+  // See TargetDisposition.
   virtual TargetDisposition GetTargetDisposition() const = 0;
 
   // Final hash of completely downloaded file, or partial hash of an interrupted
   // download; only valid if GetState() == COMPLETED or INTERRUPTED. If
   // non-empty the returned string contains a raw SHA-256 hash (i.e. not hex
   // encoded).
   virtual const std::string& GetHash() const = 0;
 
   // True if the file associated with the download has been removed by
   // external action.
@@ skipping 23 matching lines @@
 
   // Simple calculation of the amount of time remaining to completion. Fills
   // |*remaining| with the amount of time remaining if successful. Fails and
   // returns false if we do not have the number of bytes or the speed so can
   // not estimate.
   virtual bool TimeRemaining(base::TimeDelta* remaining) const = 0;
 
   // Simple speed estimate in bytes/s
   virtual int64_t CurrentSpeed() const = 0;
 
-  // Rough percent complete, -1 means we don't know (== we didn't receive a
-  // total size).
+  // Rough percent complete. Returns -1 if progress is unknown. 100 if the
+  // download is already complete.
   virtual int PercentComplete() const = 0;
 
-  // Returns true if this download has saved all of its data.
+  // Returns true if this download has saved all of its data. A download may
+  // have saved all its data but still waiting for some other process to

[clamy, 2016/10/04 14:21:30]

nit: s/still waiting/still be waiting

[asanka, 2016/10/04 14:44:55]

Done

+  // complete before the download is considered complete. E.g. A dangerous
+  // download needs to be accepted by the user before the file is renamed to its
+  // final name.
   virtual bool AllDataSaved() const = 0;
 
+  // Total number of expected bytes. Returns -1 if the total size is unknown.
   virtual int64_t GetTotalBytes() const = 0;
+
+  // Total number of bytes that have been received and written to the download
+  // file.
   virtual int64_t GetReceivedBytes() const = 0;
+
+  // Time the download was first started. This timestamp is always valid and
+  // doesn't change.
   virtual base::Time GetStartTime() const = 0;
+
+  // Time the download was marked as complete. Returns base::Time() if the
+  // download hasn't reached a completion state yet.
   virtual base::Time GetEndTime() const = 0;
 
   //    Open/Show State accessors ----------------------------------------------
 
   // Returns true if it is OK to open a folder which this file is inside.
   virtual bool CanShowInFolder() = 0;
 
   // Returns true if it is OK to open the download.
   virtual bool CanOpenDownload() = 0;
 
   // Tests if a file type should be opened automatically.
   virtual bool ShouldOpenFileBasedOnExtension() = 0;
 
   // 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;
 
   //    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;
 
   // 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_