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

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

Issue 2239133002: [Offline pages] Downloads UI: Adding bridge for issuing notifications (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@download-notifications
Patch Set: Addressing CR feedback Created 4 years, 4 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/javatests/src/org/chromium/chrome/browser/download/DownloadNotificationServiceTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/download/DownloadNotificationServiceTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/download/DownloadNotificationServiceTest.java
index 3eb53cc08cea314f808ebc94b76572be084b5033..d7937e8853917de0c8a515e50ac7f61b8a29c44c 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/download/DownloadNotificationServiceTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/download/DownloadNotificationServiceTest.java
@@ -238,14 +238,14 @@ public class DownloadNotificationServiceTest extends
DownloadNotificationService service = bindNotificationService();
String guid3 = UUID.randomUUID().toString();
- service.notifyDownloadProgress(guid3, "test", 1, 1L, 1L, true, true);
+ service.notifyDownloadProgress(guid3, "test", 1, 1L, 1L, true, true, false);
assertEquals(3, getService().getNotificationIds().size());
int lastNotificationId = getService().getLastAddedNotificationId();
Set<String> entries = DownloadManagerService.getStoredDownloadInfo(
sharedPrefs, DownloadNotificationService.PENDING_DOWNLOAD_NOTIFICATIONS);
assertEquals(3, entries.size());
- service.notifyDownloadSuccessful(guid1, "success", 100L);
+ service.notifyDownloadSuccessful(guid1, "success", 100L, false);
entries = DownloadManagerService.getStoredDownloadInfo(
sharedPrefs, DownloadNotificationService.PENDING_DOWNLOAD_NOTIFICATIONS);
assertEquals(2, entries.size());
@@ -270,7 +270,7 @@ public class DownloadNotificationServiceTest extends
startNotificationService();
DownloadNotificationService service = bindNotificationService();
String guid = UUID.randomUUID().toString();
- service.notifyDownloadSuccessful(guid, "test", 100L);
+ service.notifyDownloadSuccessful(guid, "test", 100L, false);
assertEquals(1, getService().getNotificationIds().size());
}
« no previous file with comments | « chrome/android/java_sources.gni ('k') | chrome/browser/android/offline_pages/downloads/offline_page_download_bridge.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698