From 3a35fb7967130fb4192ed96ec347fd5f7ff8995e Mon Sep 17 00:00:00 2001 From: Zhengyu Zhan Date: Fri, 10 Jul 2020 15:58:21 +0800 Subject: [PATCH] Fix data limit notification issue MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The data limited sim card has been plug out, but it’s data limit notification didn’t dismiss. It should cancel notification in function updateNotificationsNL(), but it returned. And the mActiveNotifs has clear. So it never has change to remove the notification later. Bug: 161101175 Test: com.android.server.net.NetworkPolicyManagerServiceTest All pass Change-Id: Ia08f443a88e2c9150fa2e6f474688e091d3a437b --- .../com/android/server/net/NetworkPolicyManagerService.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/services/core/java/com/android/server/net/NetworkPolicyManagerService.java b/services/core/java/com/android/server/net/NetworkPolicyManagerService.java index e8f48262fbbf9..98d04cf8457ae 100644 --- a/services/core/java/com/android/server/net/NetworkPolicyManagerService.java +++ b/services/core/java/com/android/server/net/NetworkPolicyManagerService.java @@ -1197,7 +1197,7 @@ public class NetworkPolicyManagerService extends INetworkPolicyManager.Stub { // identified carrier, which may want to manage their own notifications. This method // should be called every time the carrier config changes anyways, and there's no // reason to alert if there isn't a carrier. - return; + continue; } final boolean notifyWarning = getBooleanDefeatingNullable(config,