summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSteve Howard <showard@google.com>2010-09-10 17:21:49 -0700
committerSteve Howard <showard@google.com>2010-09-10 17:22:53 -0700
commitcae0c3610fce7336c06377698e22878d110b19d2 (patch)
tree390fcd4392311acf216d6f9b74691d531b4b0f99
parentf8ccad3e970434111b3920dc639e05ca48ca66c2 (diff)
downloadandroid_packages_providers_DownloadProvider-cae0c3610fce7336c06377698e22878d110b19d2.tar.gz
android_packages_providers_DownloadProvider-cae0c3610fce7336c06377698e22878d110b19d2.tar.bz2
android_packages_providers_DownloadProvider-cae0c3610fce7336c06377698e22878d110b19d2.zip
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
-rw-r--r--src/com/android/providers/downloads/DownloadNotification.java3
1 files changed, 1 insertions, 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<DownloadInfo> downloads) {
for (DownloadInfo download : downloads) {
if (!isCompleteAndVisible(download)) {
- return;
+ continue;
}
// Add the notifications
Notification n = new Notification();