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

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

Issue 2811803006: Add support for pulling icons for OfflineItems (Closed)
Patch Set: More findbugs Created 3 years, 8 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/MockDownloadNotificationService.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/download/MockDownloadNotificationService.java b/chrome/android/javatests/src/org/chromium/chrome/browser/download/MockDownloadNotificationService.java
index 0b260ae5b65a0ac7716302e827cb53b029a0687f..43e4e9f01683690d65e14cbb49e525748b608225 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/download/MockDownloadNotificationService.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/download/MockDownloadNotificationService.java
@@ -6,6 +6,7 @@ package org.chromium.chrome.browser.download;
import android.app.Notification;
import android.content.Context;
+import android.graphics.Bitmap;
import android.util.Pair;
import org.chromium.base.ThreadUtils;
@@ -96,13 +97,13 @@ public class MockDownloadNotificationService extends DownloadNotificationService
@Override
public int notifyDownloadSuccessful(final ContentId id, final String filePath,
final String fileName, final long systemDownloadId, final boolean isOffTheRecord,
- final boolean isSupportedMimeType, final boolean isOpenable) {
+ final boolean isSupportedMimeType, final boolean isOpenable, final Bitmap icon) {
return ThreadUtils.runOnUiThreadBlockingNoException(new Callable<Integer>() {
@Override
public Integer call() throws Exception {
return MockDownloadNotificationService.super.notifyDownloadSuccessful(id, filePath,
- fileName, systemDownloadId, isOffTheRecord, isSupportedMimeType,
- isOpenable);
+ fileName, systemDownloadId, isOffTheRecord, isSupportedMimeType, isOpenable,
+ icon);
}
});
}
@@ -111,23 +112,23 @@ public class MockDownloadNotificationService extends DownloadNotificationService
public void notifyDownloadProgress(final ContentId id, final String fileName,
final int percentage, final long bytesReceived, final long timeRemainingInMillis,
final long startTime, final boolean isOffTheRecord,
- final boolean canDownloadWhileMetered, final boolean isTransient) {
+ final boolean canDownloadWhileMetered, final boolean isTransient, final Bitmap icon) {
ThreadUtils.runOnUiThreadBlocking(new Runnable() {
@Override
public void run() {
MockDownloadNotificationService.super.notifyDownloadProgress(id, fileName,
percentage, bytesReceived, timeRemainingInMillis, startTime, isOffTheRecord,
- canDownloadWhileMetered, isTransient);
+ canDownloadWhileMetered, isTransient, icon);
}
});
}
@Override
- public void notifyDownloadFailed(final ContentId id, final String fileName) {
+ public void notifyDownloadFailed(final ContentId id, final String fileName, final Bitmap icon) {
ThreadUtils.runOnUiThreadBlocking(new Runnable() {
@Override
public void run() {
- MockDownloadNotificationService.super.notifyDownloadFailed(id, fileName);
+ MockDownloadNotificationService.super.notifyDownloadFailed(id, fileName, icon);
}
});
}

Powered by Google App Engine
This is Rietveld 408576698