From cae0c3610fce7336c06377698e22878d110b19d2 Mon Sep 17 00:00:00 2001 From: Steve Howard Date: Fri, 10 Sep 2010 17:21:49 -0700 Subject: Fix silly bug with completed notifications. This functionality isn't part of the public API, hence it lacks test coverage. Rather than adding tests, I'm inclined to leave it and hopefully we'll be able to migrate existing apps to the public API soon enough. Change-Id: I1210c6525dfb7cb12c1b8e35c04684060d3ad87e --- src/com/android/providers/downloads/DownloadNotification.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/com/android/providers/downloads/DownloadNotification.java b/src/com/android/providers/downloads/DownloadNotification.java index 2c30644b..472a5f30 100644 --- a/src/com/android/providers/downloads/DownloadNotification.java +++ b/src/com/android/providers/downloads/DownloadNotification.java @@ -22,7 +22,6 @@ import android.content.Context; import android.content.Intent; import android.net.Uri; import android.provider.Downloads; -import android.util.Log; import android.view.View; import android.widget.RemoteViews; @@ -212,7 +211,7 @@ class DownloadNotification { private void updateCompletedNotification(List downloads) { for (DownloadInfo download : downloads) { if (!isCompleteAndVisible(download)) { - return; + continue; } // Add the notifications Notification n = new Notification(); -- cgit v1.2.3