From da71ca08ef8f4aaaa5cbe25b49d166968e816e46 Mon Sep 17 00:00:00 2001 From: Fan Zhang Date: Mon, 12 Sep 2016 17:36:22 -0700 Subject: [PATCH] Parse -1 as WARNING_DISABLED in NetworkPolicyManager. Bug: 26934313 Bug: 31437693 Test: Flashed phone with new config (-1) and verified data warning is disabled out of box. Change-Id: Iabdfb9d0625698d525d7537a93b936df505bd0ab --- .../server/net/NetworkPolicyManagerService.java | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/services/core/java/com/android/server/net/NetworkPolicyManagerService.java b/services/core/java/com/android/server/net/NetworkPolicyManagerService.java index 53dbed221f8c1..3e1c529a5fc60 100644 --- a/services/core/java/com/android/server/net/NetworkPolicyManagerService.java +++ b/services/core/java/com/android/server/net/NetworkPolicyManagerService.java @@ -1442,8 +1442,14 @@ public class NetworkPolicyManagerService extends INetworkPolicyManager.Stub { + "; generating default policy"); // Build default mobile policy, and assume usage cycle starts today - final long warningBytes = mContext.getResources().getInteger( - com.android.internal.R.integer.config_networkPolicyDefaultWarning) * MB_IN_BYTES; + final int dataWarningConfig = mContext.getResources().getInteger( + com.android.internal.R.integer.config_networkPolicyDefaultWarning); + final long warningBytes; + if (dataWarningConfig == WARNING_DISABLED) { + warningBytes = WARNING_DISABLED; + } else { + warningBytes = dataWarningConfig * MB_IN_BYTES; + } final Time time = new Time(); time.setToNow();