From 9b2f2a5f041bf56ab45e5952ae4ddc4b47d0105e Mon Sep 17 00:00:00 2001
From: Jeff Sharkey <jsharkey@android.com>
Date: Tue, 18 Apr 2017 17:45:40 -0600
Subject: Create notification channels for downloads.

We tell the user about active, completed, and waiting downloads.

Test: builds, boots, downloads work
Bug: 36865435
Change-Id: Id5db029a98971a7b7d190f01eea2439120c18531
---
 .../providers/downloads/DownloadNotifier.java      | 43 ++++++++++++++++------
 1 file changed, 31 insertions(+), 12 deletions(-)

(limited to 'src')

diff --git a/src/com/android/providers/downloads/DownloadNotifier.java b/src/com/android/providers/downloads/DownloadNotifier.java
index d13bb5e2..ec1305f1 100644
--- a/src/com/android/providers/downloads/DownloadNotifier.java
+++ b/src/com/android/providers/downloads/DownloadNotifier.java
@@ -26,6 +26,7 @@ import static com.android.providers.downloads.Constants.TAG;
 
 import android.app.DownloadManager;
 import android.app.Notification;
+import android.app.NotificationChannel;
 import android.app.NotificationManager;
 import android.app.PendingIntent;
 import android.content.ContentUris;
@@ -61,6 +62,10 @@ public class DownloadNotifier {
     private static final int TYPE_WAITING = 2;
     private static final int TYPE_COMPLETE = 3;
 
+    private static final String CHANNEL_ACTIVE = "active";
+    private static final String CHANNEL_WAITING = "waiting";
+    private static final String CHANNEL_COMPLETE = "complete";
+
     private final Context mContext;
     private final NotificationManager mNotifManager;
 
@@ -89,8 +94,18 @@ public class DownloadNotifier {
 
     public DownloadNotifier(Context context) {
         mContext = context;
-        mNotifManager = (NotificationManager) context.getSystemService(
-                Context.NOTIFICATION_SERVICE);
+        mNotifManager = context.getSystemService(NotificationManager.class);
+
+        // Ensure that all our channels are ready to use
+        mNotifManager.createNotificationChannel(new NotificationChannel(CHANNEL_ACTIVE,
+                context.getText(R.string.download_running),
+                NotificationManager.IMPORTANCE_LOW));
+        mNotifManager.createNotificationChannel(new NotificationChannel(CHANNEL_WAITING,
+                context.getText(R.string.download_queued),
+                NotificationManager.IMPORTANCE_DEFAULT));
+        mNotifManager.createNotificationChannel(new NotificationChannel(CHANNEL_COMPLETE,
+                context.getText(com.android.internal.R.string.done_label),
+                NotificationManager.IMPORTANCE_DEFAULT));
     }
 
     public void init() {
@@ -178,7 +193,20 @@ public class DownloadNotifier {
             final IntArray cluster = clustered.valueAt(i);
             final int type = getNotificationTagType(tag);
 
-            final Notification.Builder builder = new Notification.Builder(mContext);
+            final Notification.Builder builder;
+            if (type == TYPE_ACTIVE) {
+                builder = new Notification.Builder(mContext, CHANNEL_ACTIVE);
+                builder.setSmallIcon(android.R.drawable.stat_sys_download);
+            } else if (type == TYPE_WAITING) {
+                builder = new Notification.Builder(mContext, CHANNEL_WAITING);
+                builder.setSmallIcon(android.R.drawable.stat_sys_warning);
+            } else if (type == TYPE_COMPLETE) {
+                builder = new Notification.Builder(mContext, CHANNEL_COMPLETE);
+                builder.setSmallIcon(android.R.drawable.stat_sys_download_done);
+            } else {
+                continue;
+            }
+
             builder.setColor(res.getColor(
                     com.android.internal.R.color.system_notification_accent_color));
 
@@ -192,15 +220,6 @@ public class DownloadNotifier {
             }
             builder.setWhen(firstShown);
 
-            // Show relevant icon
-            if (type == TYPE_ACTIVE) {
-                builder.setSmallIcon(android.R.drawable.stat_sys_download);
-            } else if (type == TYPE_WAITING) {
-                builder.setSmallIcon(android.R.drawable.stat_sys_warning);
-            } else if (type == TYPE_COMPLETE) {
-                builder.setSmallIcon(android.R.drawable.stat_sys_download_done);
-            }
-
             // Build action intents
             if (type == TYPE_ACTIVE || type == TYPE_WAITING) {
                 final long[] downloadIds = getDownloadIds(cursor, cluster);
-- 
cgit v1.2.3