Fixed the set/getAllowedNetworkTypes debug messages
Made the debug message more readable.
Fix: 265535191
Test: Manual
Change-Id: If0c16c2bb48f2ab0eaea2f1d7ef0dbb834685b43
diff --git a/src/com/android/phone/PhoneInterfaceManager.java b/src/com/android/phone/PhoneInterfaceManager.java
index 01bbe4f..4d467ab 100644
--- a/src/com/android/phone/PhoneInterfaceManager.java
+++ b/src/com/android/phone/PhoneInterfaceManager.java
@@ -6727,10 +6727,11 @@
throw new SecurityException(
"setAllowedNetworkTypesForReason cannot be called with carrier privileges for"
+ " reason "
- + reason);
+ + TelephonyManager.allowedNetworkTypesReasonToString(reason));
}
if (!TelephonyManager.isValidAllowedNetworkTypesReason(reason)) {
- loge("setAllowedNetworkTypesForReason: Invalid allowed network type reason: " + reason);
+ loge("setAllowedNetworkTypesForReason: Invalid allowed network type reason: "
+ + TelephonyManager.allowedNetworkTypesReasonToString(reason));
return false;
}
if (!SubscriptionManager.isUsableSubscriptionId(subId)) {
@@ -6738,8 +6739,10 @@
return false;
}
- log("setAllowedNetworkTypesForReason: subId=" + subId + ", reason=" + reason + " value: "
- + TelephonyManager.convertNetworkTypeBitmaskToString(allowedNetworkTypes));
+ log("setAllowedNetworkTypesForReason: subId=" + subId + ", reason="
+ + TelephonyManager.allowedNetworkTypesReasonToString(reason) + ", network types: "
+ + TelephonyManager.convertNetworkTypeBitmaskToString(allowedNetworkTypes)
+ + ", callingPackage=" + getCurrentPackageName());
Phone phone = getPhone(subId);
if (phone == null) {
@@ -6747,7 +6750,9 @@
}
if (allowedNetworkTypes == phone.getAllowedNetworkTypes(reason)) {
- log("setAllowedNetworkTypesForReason: " + reason + "does not change value");
+ log("setAllowedNetworkTypesForReason: "
+ + TelephonyManager.allowedNetworkTypesReasonToString(reason)
+ + " already has the specified network types.");
return true;
}