[Downloads] Introduce GUIDs for downloads.

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 d3a7798a019b6f6976de26e6d3cdaaaaebc9e344..56e6e4ed1e915059fced268b15e77f3b1832cd45 100644
--- a/content/public/browser/download_item.h
+++ b/content/public/browser/download_item.h
@@ -154,7 +154,16 @@ class CONTENT_EXPORT DownloadItem : public base::SupportsUserData {
 
   // 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.
   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.
+  virtual const std::string& GetGuid() const = 0;
+
   virtual DownloadState GetState() const = 0;

[svaldez, 2016/03/14 16:59:50]

Consider adding a comment here.

[asanka, 2016/03/14 19:18:33]

Done.

 
   // Returns the most recent interrupt reason for this download. Returns