Adding status info to DownloadUIItem and piping it through.

components/offline_pages/core/downloads/download_ui_item.h

Index: components/offline_pages/core/downloads/download_ui_item.h
diff --git a/components/offline_pages/core/downloads/download_ui_item.h b/components/offline_pages/core/downloads/download_ui_item.h
index aea7cf5ccebb9b8e8dc8d698d049b4c4051117f6..4b81f879edc3cb838ef9fad5c2842343ff218257 100644
--- a/components/offline_pages/core/downloads/download_ui_item.h
+++ b/components/offline_pages/core/downloads/download_ui_item.h
@@ -18,20 +18,31 @@ namespace offline_pages {
 struct OfflinePageItem;
 class SavePageRequest;
 
+// The abstract "download item" that may be a media file, a web page (together
+// with all the resources) or a PWA web package. This is a data bag that exposes
+// only bits potentially visible by the user, not the internal data.
 struct DownloadUIItem {
  public:
+  // GENERATED_JAVA_ENUM_PACKAGE: org.chromium.components.offlinepages.downloads
+  enum DownloadState { PENDING = 0, IN_PROGRESS = 1, PAUSED = 2, COMPLETE = 3 };
+
   DownloadUIItem();
   explicit DownloadUIItem(const OfflinePageItem& page);
   explicit DownloadUIItem(const SavePageRequest& request);
   DownloadUIItem(const DownloadUIItem& other);
   ~DownloadUIItem();
 
-  // Unique id.
+  // Unique id. It is filled with the offline_id, which should be ok

[fgorski, 2017/01/30 21:39:51]

nit: ID

[Dmitry Titov, 2017/02/08 00:45:08]

Done.
Also reverted the rest of the comment since it is not yet using offlien_id
there.

+  // since there is no cross-exchange of these items between download backends.
   std::string guid;
 
   // The URL of the captured page.
   GURL url;
 
+  DownloadState download_state;
+
+  int64_t download_progress_bytes;
+
   // The Title of the captured page, if any. It can be empty string either
   // because the page is not yet fully loaded, or because it doesn't have any.
   base::string16 title;