diff options
author | Roshan Pius <rpius@google.com> | 2019-12-09 10:37:47 -0800 |
---|---|---|
committer | Roshan Pius <rpius@google.com> | 2019-12-09 11:20:01 -0800 |
commit | 6bec9adffcfc7bb19d13121dac1451745c49f00d (patch) | |
tree | a2e9f17c1688020f178dd9b493f2fc78c642cae2 | |
parent | 64a3f5922c9440e170bb1ed32633cfab8c97ed62 (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.java | 2 |
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); |