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

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

Issue 2789083002: Properly clean up notification on download open (Closed)
Patch Set: Created 3 years, 9 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/java/src/org/chromium/chrome/browser/download/DownloadSnackbarController.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadSnackbarController.java b/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadSnackbarController.java
index 401c1dbe7c4ae6e0f85ff5f4d64cba0db8e26f6b..93629e0812b84f2cbfc208f920f334de255faafd 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadSnackbarController.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadSnackbarController.java
@@ -5,7 +5,6 @@
package org.chromium.chrome.browser.download;
import android.app.Activity;
-import android.app.NotificationManager;
import android.content.Context;
import org.chromium.base.ApplicationStatus;
@@ -54,10 +53,8 @@ public class DownloadSnackbarController implements SnackbarManager.SnackbarContr
DownloadManagerService manager = DownloadManagerService.getDownloadManagerService(mContext);
manager.openDownloadedContent(download.downloadInfo, download.systemDownloadId);
if (download.notificationId != INVALID_NOTIFICATION_ID) {
- NotificationManager notificationManager =
- (NotificationManager) mContext.getSystemService(Context.NOTIFICATION_SERVICE);
- notificationManager.cancel(
- DownloadNotificationService.NOTIFICATION_NAMESPACE, download.notificationId);
+ manager.getDownloadNotifier().removeDownloadNotification(
+ download.notificationId, download.downloadInfo);
}
}

Powered by Google App Engine
This is Rietveld 408576698