Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(311)

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

Issue 2631933002: Adding status info to DownloadUIItem and piping it through. (Closed)
Patch Set: fix chrome_test_java Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/java/src/org/chromium/chrome/browser/offlinepages/downloads/OfflinePageDownloadItem.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/downloads/OfflinePageDownloadItem.java b/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/downloads/OfflinePageDownloadItem.java
index 3ac4657dabdb9cb95e2462408573cde1dc765902..7c542da357cbdfb08197f07734113197f47be51f 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/downloads/OfflinePageDownloadItem.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/offlinepages/downloads/OfflinePageDownloadItem.java
@@ -4,20 +4,26 @@
package org.chromium.chrome.browser.offlinepages.downloads;
+import org.chromium.components.offlinepages.downloads.DownloadState;
+
/** Class representing offline page or save page request to downloads UI. */
public class OfflinePageDownloadItem {
private final String mUrl;
+ private final int mDownloadState;
+ private final long mDownloadProgressBytes;
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,
+ public OfflinePageDownloadItem(String guid, String url, int downloadState,
+ long downloadProgressBytes, String title, String targetPath,
long startTimeMs, long totalBytes) {
mGuid = guid;
mUrl = url;
+ mDownloadState = downloadState;
+ mDownloadProgressBytes = downloadProgressBytes;
mTitle = title;
mTargetPath = targetPath;
mStartTimeMs = startTimeMs;
@@ -34,6 +40,18 @@ public class OfflinePageDownloadItem {
return mUrl;
}
+ /** @return DownloadState value. */
+ public int getDownloadState() {
+ return mDownloadState;
+ }
+
+ /** @return current download progress while the item is downloaded.
gone 2017/02/09 23:56:57 If it doesn't fit on one line, /** sits on a line
Dmitry Titov 2017/02/10 02:04:39 Done.
+ Returns 0 if the item is not currently downloading. */
+ public long getDownloadProgressBytes() {
+ if (mDownloadState != DownloadState.IN_PROGRESS) return 0;
+ return mDownloadProgressBytes;
+ }
+
/** @return Title of the page. */
public String getTitle() {
return mTitle;

Powered by Google App Engine
This is Rietveld 408576698