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

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

Issue 2493543003: [Downloads] Pass more DownloadItem state to java (Closed)
Patch Set: Removed dependency 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/javatests/src/org/chromium/chrome/browser/download/ui/DownloadHistoryAdapterTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/download/ui/DownloadHistoryAdapterTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/download/ui/DownloadHistoryAdapterTest.java
index 0a3767b2d71dda1e89c6bb0eac2d9d4ec46369db..df804fcbf7a78227d8729e25f5a4690355de9d51 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/download/ui/DownloadHistoryAdapterTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/download/ui/DownloadHistoryAdapterTest.java
@@ -17,7 +17,6 @@ import org.chromium.chrome.browser.download.ui.StubbedProvider.StubbedDownloadDe
import org.chromium.chrome.browser.download.ui.StubbedProvider.StubbedOfflinePageDelegate;
import org.chromium.chrome.browser.offlinepages.downloads.OfflinePageDownloadItem;
import org.chromium.content.browser.test.NativeLibraryTestBase;
-import org.chromium.content_public.browser.DownloadState;
/**
* Tests a DownloadHistoryAdapter that is isolated from the real bridges.
@@ -159,7 +158,7 @@ public class DownloadHistoryAdapterTest extends NativeLibraryTestBase {
// Add the first item.
assertEquals(1, mObserver.onChangedCallback.getCallCount());
DownloadItem item0 = StubbedProvider.createDownloadItem(0, "19840116 12:00");
- mAdapter.onDownloadItemUpdated(item0, false, DownloadState.COMPLETE);
+ mAdapter.onDownloadItemUpdated(item0, false);
mObserver.onChangedCallback.waitForCallback(1);
checkAdapterContents(null, item0);
assertEquals(1, mAdapter.getTotalDownloadSize());
@@ -167,7 +166,7 @@ public class DownloadHistoryAdapterTest extends NativeLibraryTestBase {
// Add a second item with a different date.
assertEquals(2, mObserver.onChangedCallback.getCallCount());
DownloadItem item1 = StubbedProvider.createDownloadItem(1, "19840117 12:00");
- mAdapter.onDownloadItemUpdated(item1, false, DownloadState.COMPLETE);
+ mAdapter.onDownloadItemUpdated(item1, false);
mObserver.onChangedCallback.waitForCallback(2);
checkAdapterContents(null, item1, null, item0);
assertEquals(11, mAdapter.getTotalDownloadSize());
@@ -175,7 +174,7 @@ public class DownloadHistoryAdapterTest extends NativeLibraryTestBase {
// Add a third item with the same date as the second item.
assertEquals(3, mObserver.onChangedCallback.getCallCount());
DownloadItem item2 = StubbedProvider.createDownloadItem(2, "19840117 18:00");
- mAdapter.onDownloadItemUpdated(item2, false, DownloadState.COMPLETE);
+ mAdapter.onDownloadItemUpdated(item2, false);
mObserver.onChangedCallback.waitForCallback(3);
checkAdapterContents(null, item2, item1, null, item0);
assertEquals(111, mAdapter.getTotalDownloadSize());
@@ -183,7 +182,7 @@ public class DownloadHistoryAdapterTest extends NativeLibraryTestBase {
// An item with the same download ID as the second item should just update the old one.
assertEquals(4, mObserver.onChangedCallback.getCallCount());
DownloadItem item3 = StubbedProvider.createDownloadItem(2, "19840117 18:00");
- mAdapter.onDownloadItemUpdated(item3, false, DownloadState.COMPLETE);
+ mAdapter.onDownloadItemUpdated(item3, false);
mObserver.onChangedCallback.waitForCallback(4);
checkAdapterContents(null, item3, item1, null, item0);
assertEquals(111, mAdapter.getTotalDownloadSize());

Powered by Google App Engine
This is Rietveld 408576698