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 9f6723678d1277d37f10de1cab07543e00f4b7b4..7f3b06cd8237c6787871bcb3162e72764cdb57ca 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 |
@@ -62,7 +62,7 @@ public class DownloadHistoryAdapterTest extends NativeLibraryTestBase { |
} |
}); |
mDownloadDelegate.addCallback.waitForCallback(0); |
- mObserver.onChangedCallback.waitForCallback(callCount, showOffTheRecord ? 3 : 2); |
+ mObserver.onChangedCallback.waitForCallback(callCount, 1); |
} |
/** Nothing downloaded, nothing shown. */ |
@@ -156,49 +156,49 @@ public class DownloadHistoryAdapterTest extends NativeLibraryTestBase { |
assertEquals(0, mAdapter.getTotalDownloadSize()); |
// Add the first item. |
- assertEquals(2, mObserver.onChangedCallback.getCallCount()); |
+ assertEquals(1, mObserver.onChangedCallback.getCallCount()); |
DownloadItem item0 = StubbedProvider.createDownloadItem(0, "19840116 12:00"); |
mAdapter.onDownloadItemUpdated(item0, false); |
- mObserver.onChangedCallback.waitForCallback(2); |
+ mObserver.onChangedCallback.waitForCallback(1); |
checkAdapterContents(null, item0); |
assertEquals(1, mAdapter.getTotalDownloadSize()); |
// Add a second item with a different date. |
- assertEquals(3, mObserver.onChangedCallback.getCallCount()); |
+ assertEquals(2, mObserver.onChangedCallback.getCallCount()); |
DownloadItem item1 = StubbedProvider.createDownloadItem(1, "19840117 12:00"); |
mAdapter.onDownloadItemUpdated(item1, false); |
- mObserver.onChangedCallback.waitForCallback(3); |
+ mObserver.onChangedCallback.waitForCallback(2); |
checkAdapterContents(null, item1, null, item0); |
assertEquals(11, mAdapter.getTotalDownloadSize()); |
// Add a third item with the same date as the second item. |
- assertEquals(4, mObserver.onChangedCallback.getCallCount()); |
+ assertEquals(3, mObserver.onChangedCallback.getCallCount()); |
DownloadItem item2 = StubbedProvider.createDownloadItem(2, "19840117 18:00"); |
mAdapter.onDownloadItemUpdated(item2, false); |
- mObserver.onChangedCallback.waitForCallback(4); |
+ mObserver.onChangedCallback.waitForCallback(3); |
checkAdapterContents(null, item2, item1, null, item0); |
assertEquals(111, mAdapter.getTotalDownloadSize()); |
// An item with the same download ID as the second item should just update the old one. |
- assertEquals(5, mObserver.onChangedCallback.getCallCount()); |
+ assertEquals(4, mObserver.onChangedCallback.getCallCount()); |
DownloadItem item3 = StubbedProvider.createDownloadItem(2, "19840117 18:00"); |
mAdapter.onDownloadItemUpdated(item3, false); |
- mObserver.onChangedCallback.waitForCallback(5); |
+ mObserver.onChangedCallback.waitForCallback(4); |
checkAdapterContents(null, item3, item1, null, item0); |
assertEquals(111, mAdapter.getTotalDownloadSize()); |
// Throw on a new OfflinePageItem. |
- assertEquals(6, mObserver.onChangedCallback.getCallCount()); |
+ assertEquals(5, mObserver.onChangedCallback.getCallCount()); |
OfflinePageDownloadItem item4 = StubbedProvider.createOfflineItem(0, "19840117 19:00"); |
mOfflineDelegate.observer.onItemAdded(item4); |
- mObserver.onChangedCallback.waitForCallback(6); |
+ mObserver.onChangedCallback.waitForCallback(5); |
checkAdapterContents(null, item4, item3, item1, null, item0); |
// Update the existing OfflinePageItem. |
- assertEquals(7, mObserver.onChangedCallback.getCallCount()); |
+ assertEquals(6, mObserver.onChangedCallback.getCallCount()); |
OfflinePageDownloadItem item5 = StubbedProvider.createOfflineItem(0, "19840117 19:00"); |
mOfflineDelegate.observer.onItemUpdated(item5); |
- mObserver.onChangedCallback.waitForCallback(7); |
+ mObserver.onChangedCallback.waitForCallback(6); |
checkAdapterContents(null, item5, item3, item1, null, item0); |
} |
@@ -218,23 +218,23 @@ public class DownloadHistoryAdapterTest extends NativeLibraryTestBase { |
assertEquals(100011, mAdapter.getTotalDownloadSize()); |
// Remove an item from the date bucket with two items. |
- assertEquals(3, mObserver.onChangedCallback.getCallCount()); |
+ assertEquals(1, mObserver.onChangedCallback.getCallCount()); |
mAdapter.onDownloadItemRemoved(offTheRecordItem.getId(), true); |
- mObserver.onChangedCallback.waitForCallback(3); |
+ mObserver.onChangedCallback.waitForCallback(1); |
checkAdapterContents(null, offlineItem, null, regularItem); |
assertEquals(100001, mAdapter.getTotalDownloadSize()); |
// Remove an item from the second bucket, which removes the bucket entirely. |
- assertEquals(4, mObserver.onChangedCallback.getCallCount()); |
+ assertEquals(2, mObserver.onChangedCallback.getCallCount()); |
mOfflineDelegate.observer.onItemDeleted(offlineItem.getGuid()); |
- mObserver.onChangedCallback.waitForCallback(4); |
+ mObserver.onChangedCallback.waitForCallback(2); |
checkAdapterContents(null, regularItem); |
assertEquals(1, mAdapter.getTotalDownloadSize()); |
// Remove the last item in the list. |
- assertEquals(5, mObserver.onChangedCallback.getCallCount()); |
+ assertEquals(3, mObserver.onChangedCallback.getCallCount()); |
mAdapter.onDownloadItemRemoved(regularItem.getId(), false); |
- mObserver.onChangedCallback.waitForCallback(5); |
+ mObserver.onChangedCallback.waitForCallback(3); |
assertEquals(0, mAdapter.getItemCount()); |
assertEquals(0, mAdapter.getTotalDownloadSize()); |
} |