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

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

Issue 2502113002: [Download Home] More cleaning (Closed)
Patch Set: Comment Created 4 years, 1 month 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/DeletedFileTracker.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DeletedFileTracker.java b/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DeletedFileTracker.java
index 194c249734d61dd727e82457f946378254dd5495..1d8a02cee46c0155c6b27e306f8a0ce3bb40f5d8 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DeletedFileTracker.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DeletedFileTracker.java
@@ -4,6 +4,8 @@
package org.chromium.chrome.browser.download.ui;
+import org.chromium.chrome.browser.download.ui.DownloadHistoryItemWrapper.DownloadItemWrapper;
+
import java.util.HashSet;
import java.util.Set;
import java.util.concurrent.atomic.AtomicInteger;
@@ -34,14 +36,16 @@ class DeletedFileTracker {
}
/** Add a new item to the tracker. */
- void add(String id, boolean isOffTheRecord) {
- Set<String> items = isOffTheRecord ? mIncognitoItems : mRegularItems;
- items.add(id);
+ void add(DownloadHistoryItemWrapper wrapper) {
+ if (!(wrapper instanceof DownloadItemWrapper)) return;
+ Set<String> items = wrapper.isOffTheRecord() ? mIncognitoItems : mRegularItems;
+ items.add(wrapper.getId());
}
/** Checks if an item is in the tracker. */
- boolean contains(String id, boolean isOffTheRecord) {
- Set<String> items = isOffTheRecord ? mIncognitoItems : mRegularItems;
- return items.contains(id);
+ boolean contains(DownloadHistoryItemWrapper wrapper) {
+ if (!(wrapper instanceof DownloadItemWrapper)) return false;
+ Set<String> items = wrapper.isOffTheRecord() ? mIncognitoItems : mRegularItems;
+ return items.contains(wrapper.getId());
}
}

Powered by Google App Engine
This is Rietveld 408576698