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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryItemWrapper.java

Issue 2704853002: Downloads Home - Offline page progress support (Closed)
Patch Set: Fixed wrong function call 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/download/ui/DownloadHistoryItemWrapper.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryItemWrapper.java b/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryItemWrapper.java
index 8cb1cdd850410caf6ff84fa8b4a5fbc403376436..9b483512360b617a872da37ae33aee726de83260 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryItemWrapper.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryItemWrapper.java
@@ -517,17 +517,17 @@ public abstract class DownloadHistoryItemWrapper extends TimedItem {
@Override
public void cancel() {
- assert false;
+ mBackendProvider.getOfflinePageBridge().cancelDownload(getId());
}
@Override
public void pause() {
- assert false;
+ mBackendProvider.getOfflinePageBridge().pauseDownload(getId());
}
@Override
public void resume() {
- assert false;
+ mBackendProvider.getOfflinePageBridge().resumeDownload(getId());
}
@Override
@@ -549,12 +549,14 @@ public abstract class DownloadHistoryItemWrapper extends TimedItem {
@Override
public boolean isComplete() {
- return true;
+ return mItem.getDownloadState()
+ == org.chromium.components.offlinepages.downloads.DownloadState.COMPLETE;
}
@Override
public boolean isPaused() {
- return false;
+ return mItem.getDownloadState()
+ == org.chromium.components.offlinepages.downloads.DownloadState.PAUSED;
}
}
}

Powered by Google App Engine
This is Rietveld 408576698