summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRoshan Pius <rpius@google.com>2019-12-09 10:37:47 -0800
committerRoshan Pius <rpius@google.com>2019-12-09 11:20:01 -0800
commit6bec9adffcfc7bb19d13121dac1451745c49f00d (patch)
treea2e9f17c1688020f178dd9b493f2fc78c642cae2
parent64a3f5922c9440e170bb1ed32633cfab8c97ed62 (diff)
WifiService: Remove usage of NotificationChannel.setBlockableSystem
For non-signature apps, the default behavior is "blockable". So, this @hide call is redundant. For non-mainline devices, the network* channels used by wifi are the "common" networking channels. On AOSP, they are created by |SystemNotificationChannels| and marked as "blockable". So, unless the OEM changes the AOSP version of |SystemNotificationChannels|, they will continue to get the correct behavior. Bug: 145921908 Test: Compiles Change-Id: If705203d92a5f9ea850e42cf0e43efb970295927
-rw-r--r--service/java/com/android/server/wifi/WifiService.java2
1 files changed, 0 insertions, 2 deletions
diff --git a/service/java/com/android/server/wifi/WifiService.java b/service/java/com/android/server/wifi/WifiService.java
index 34f1a76b7..a6a32bb6e 100644
--- a/service/java/com/android/server/wifi/WifiService.java
+++ b/service/java/com/android/server/wifi/WifiService.java
@@ -94,7 +94,6 @@ public final class WifiService extends SystemService {
ctx.getResources().getString(
android.R.string.notification_channel_network_alerts),
NotificationManager.IMPORTANCE_HIGH);
- networkAlertsChannel.setBlockableSystem(true);
channelsList.add(networkAlertsChannel);
final NotificationChannel networkAvailable = new NotificationChannel(
@@ -102,7 +101,6 @@ public final class WifiService extends SystemService {
ctx.getResources().getString(
android.R.string.notification_channel_network_available),
NotificationManager.IMPORTANCE_LOW);
- networkAvailable.setBlockableSystem(true);
channelsList.add(networkAvailable);
nm.createNotificationChannels(channelsList);