Adding status info to DownloadUIItem and piping it through.

chrome/android/java/src/org/chromium/chrome/browser/offlinepages/downloads/OfflinePageDownloadItem.java

 // Copyright 2016 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.
 
 package org.chromium.chrome.browser.offlinepages.downloads;
 
 /** Class representing offline page or save page request to downloads UI. */
 public class OfflinePageDownloadItem {
+    /** Download State, in sync with DownloadUIItem::DownloadState */

[fgorski, 2017/01/20 17:03:01]

There is a way to enforce that using compiler. For that, please remove these and
replace with"
import org.chromium.components.offlinepages.downloads.DownloadState;

After you apply the comment in
https://codereview.chromium.org/2631933002/diff/20001/components/offline_page...

[gone, 2017/01/20 18:40:01]

Alternatively, can you just use
org.chromium.content_public.browser.DownloadState?  Looks like you're trying to
match those states.

[Dmitry Titov, 2017/01/27 04:26:24]

Can not include content/public/browser stuff in all places I need (components),
bad dependency. So going with Filip's advice.

+    private static final int PENDING = 0;
+    private static final int DOWNLOADING = 1;
+    private static final int PAUSED = 2;
+    private static final int COMPLETED = 3;
+
     private final String mUrl;
+    private final int mDownloadState;
+    private final long mDownloadProgress;
+    private final long mDownloadProgressMax;
     private final String mTitle;
     private final String mGuid;
     private final String mTargetPath;
     private final long mStartTimeMs;
     private final long mTotalBytes;
 
     public OfflinePageDownloadItem(
-            String guid, String url, String title, String targetPath,
+            String guid, String url, int downloadState, long downloadProgress,
+            long downloadProgressMax, String title, String targetPath,
             long startTimeMs, long totalBytes) {
         mGuid = guid;
         mUrl = url;
+        mDownloadState = downloadState;
+        mDownloadProgress = downloadProgress;
+        mDownloadProgressMax = downloadProgressMax;
         mTitle = title;
         mTargetPath = targetPath;
         mStartTimeMs = startTimeMs;
         mTotalBytes = totalBytes;
     }
 
     /** @return GUID identifying the item. */
     public String getGuid() {
         return mGuid;
     }
 
     /** @return URL related to the item. */
     public String getUrl() {
         return mUrl;
     }
 
+    /** @return DownloadState value. */
+    public int downloadState() {
+        return mDownloadState;
+    }
+
+    /** @return current download progress */
+    public long downloadPreogress() {

[dewittj, 2017/01/18 19:24:07]

s/Preogress/Progress/ also in next function

[Dmitry Titov, 2017/01/27 04:26:24]

Done.

+        return mDownloadProgress;
+    }
+
+    /** @return current download progress */
+    public long downloadPreogressMax() {
+        return mDownloadProgressMax;
+    }
+
     /** @return Title of the page. */
     public String getTitle() {
         return mTitle;
     }
 
     /** @return Path to the offline item on the disk. */
     // TODO(fgorski): Title would be more meaningful to show in the Download UI, where the local
     // path is shown right now.
     public String getTargetPath() {
         return mTargetPath;
     }
 
     /** @return Start time of the item, corresponding to when the offline page was saved. */
     public long getStartTimeMs() {
         return mStartTimeMs;
     }
 
     /** @return Size of the offline archive in bytes. */
     public long getTotalBytes() {
         return mTotalBytes;
     }
 }