commit 297396926ffdf85ea32964436b5e9c4b1e86b0c3
author Malcolm Chen <refuhoo@google.com> Thu Apr 18 13:51:02 2019 -0700
committer Malcolm Chen <refuhoo@google.com> Thu Apr 18 19:28:42 2019 -0700
tree aa72bd36e0a52187e3ea45560c75d76744f9cda0
parent 460810da7a0d22cf037f865ae04f15db117ac3cc

Adding @hide API to return whether an APN is metered or not.

In addition, make isMmsDataEnabled more generic:
isDataEnabledForApn

Bug: 128616035
Test: manual
Change-Id: Ia2f2a1c57ae34df57f1174368ec75bdf8a7eecbc
Merged-In: Ia2f2a1c57ae34df57f1174368ec75bdf8a7eecbc

src/com/android/phone/PhoneInterfaceManager.java

diff --git a/src/com/android/phone/PhoneInterfaceManager.java b/src/com/android/phone/PhoneInterfaceManager.java
index 8293802..46c2847 100755
--- a/src/com/android/phone/PhoneInterfaceManager.java
+++ b/src/com/android/phone/PhoneInterfaceManager.java
@@ -17,7 +17,6 @@
 package com.android.phone;
 
 import static android.content.pm.PackageManager.PERMISSION_GRANTED;
-import static android.telephony.data.ApnSetting.TYPE_MMS;
 
 import static com.android.internal.telephony.PhoneConstants.SUBSCRIPTION_KEY;
 
@@ -6937,16 +6936,24 @@
     }
 
     /**
-     *  Mms is allowed if
+     * Return whether data is enabled for certain APN type. This will tell if framework will accept
+     * corresponding network requests on a subId.
+     *
+     *  Data is enabled if:
      *  1) user data is turned on, or
-     *  2) Mms is un-metered, or
-     *  3) alwaysAllowMms setting is turned on.
+     *  2) APN is un-metered for this subscription, or
+     *  3) APN type is whitelisted. E.g. MMS is whitelisted if
+     *  {@link SubscriptionManager#setAlwaysAllowMmsData} is turned on.
+     *
+     * @return whether data is allowed for a apn type.
+     *
+     * @hide
      */
     @Override
-    public boolean isMmsDataEnabled(int subId, String callingPackage) {
+    public boolean isDataEnabledForApn(int apnType, int subId, String callingPackage) {
         if (!TelephonyPermissions.checkCallingOrSelfReadPhoneState(
-                mApp, subId, callingPackage, "isMmsDataEnabled")) {
-            throw new SecurityException("Needs READ_PHONE_STATE for isMmsDataEnabled");
+                mApp, subId, callingPackage, "isDataEnabledForApn")) {
+            throw new SecurityException("Needs READ_PHONE_STATE for isDataEnabledForApn");
         }
 
         // Now that all security checks passes, perform the operation as ourselves.
@@ -6956,8 +6963,25 @@
             if (phone == null) return false;
 
             boolean isMetered = ApnSettingUtils.isMeteredApnType(ApnSetting.getApnTypeString(
-                    TYPE_MMS), phone);
-            return !isMetered || phone.getDataEnabledSettings().isDataEnabled(TYPE_MMS);
+                    apnType), phone);
+            return !isMetered || phone.getDataEnabledSettings().isDataEnabled(apnType);
+        } finally {
+            Binder.restoreCallingIdentity(identity);
+        }
+    }
+
+    @Override
+    public boolean isApnMetered(int apnType, int subId) {
+        enforceReadPrivilegedPermission("isApnMetered");
+
+        // Now that all security checks passes, perform the operation as ourselves.
+        final long identity = Binder.clearCallingIdentity();
+        try {
+            Phone phone = getPhone(subId);
+            if (phone == null) return true; // By default return true.
+
+            return ApnSettingUtils.isMeteredApnType(ApnSetting.getApnTypeString(
+                    apnType), phone);
         } finally {
             Binder.restoreCallingIdentity(identity);
         }