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

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

Issue 2320693003: [Download Home] Expose Java side to DownloadItem states (Closed)
Patch Set: Rebased Created 4 years, 3 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/DownloadHistoryAdapter.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryAdapter.java b/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryAdapter.java
index eaaaae16766faf12ee9cba8339d0e11f2b1acb42..c6bd5a5ef9b17932875a914390b2e9492ec59929 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryAdapter.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryAdapter.java
@@ -24,6 +24,7 @@ import org.chromium.chrome.browser.offlinepages.downloads.OfflinePageDownloadBri
import org.chromium.chrome.browser.offlinepages.downloads.OfflinePageDownloadItem;
import org.chromium.chrome.browser.widget.DateDividedAdapter;
import org.chromium.chrome.browser.widget.selection.SelectionDelegate;
+import org.chromium.content_public.browser.DownloadState;
import java.util.ArrayList;
import java.util.HashMap;
@@ -174,9 +175,12 @@ public class DownloadHistoryAdapter extends DateDividedAdapter implements Downlo
/**
* Updates the list when new information about a download comes in.
*/
- public void onDownloadItemUpdated(DownloadItem item, boolean isOffTheRecord) {
+ public void onDownloadItemUpdated(DownloadItem item, boolean isOffTheRecord, int state) {
if (isOffTheRecord && !mShowOffTheRecord) return;
+ // The adapter currently only cares about completion events.
+ if (state != DownloadState.COMPLETE) return;
+
List<DownloadItemWrapper> list = getDownloadItemList(isOffTheRecord);
int index = findItemIndex(list, item.getId());

Powered by Google App Engine
This is Rietveld 408576698