summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Greg Kaiser <gkaiser@google.com> 2020-12-22 19:36:50 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> 2020-12-22 19:36:50 +0000
commit469e5b33177a7d252c7e74a4112ade3de95f11c9 (patch)
treeccef7d568f53f8cb47d87cee0df6a01e7424a955
parenteff887b71494c8f141b772da8249364eaf00ed32 (diff)
parent8ce575b256177508d35604f95d5db8c03b09dbfd (diff)
Merge "Revert "Fix permission on TM#isDataEnabled""
-rw-r--r--core/api/current.txt2
-rw-r--r--telephony/java/android/telephony/TelephonyManager.java7
2 files changed, 4 insertions, 5 deletions
diff --git a/core/api/current.txt b/core/api/current.txt
index c79d4b8050f4..74d313c636b7 100644
--- a/core/api/current.txt
+++ b/core/api/current.txt
@@ -41765,7 +41765,7 @@ package android.telephony {
method @Deprecated public String iccTransmitApduLogicalChannel(int, int, int, int, int, int, String);
method public boolean isConcurrentVoiceAndDataSupported();
method @RequiresPermission(anyOf={android.Manifest.permission.ACCESS_NETWORK_STATE, android.Manifest.permission.READ_PHONE_STATE, "android.permission.READ_PRIVILEGED_PHONE_STATE"}) public boolean isDataConnectionAllowed();
- method @RequiresPermission(anyOf={android.Manifest.permission.ACCESS_NETWORK_STATE, android.Manifest.permission.MODIFY_PHONE_STATE, android.Manifest.permission.READ_PHONE_STATE}) public boolean isDataEnabled();
+ method @RequiresPermission(anyOf={android.Manifest.permission.ACCESS_NETWORK_STATE, android.Manifest.permission.MODIFY_PHONE_STATE}) public boolean isDataEnabled();
method @RequiresPermission(anyOf={android.Manifest.permission.ACCESS_NETWORK_STATE, android.Manifest.permission.READ_PHONE_STATE}) public boolean isDataEnabledForReason(int);
method @RequiresPermission(anyOf={android.Manifest.permission.ACCESS_NETWORK_STATE, android.Manifest.permission.READ_PHONE_STATE}) public boolean isDataRoamingEnabled();
method public boolean isEmergencyNumber(@NonNull String);
diff --git a/telephony/java/android/telephony/TelephonyManager.java b/telephony/java/android/telephony/TelephonyManager.java
index 26fd99de0a04..dd4b6428901c 100644
--- a/telephony/java/android/telephony/TelephonyManager.java
+++ b/telephony/java/android/telephony/TelephonyManager.java
@@ -9338,10 +9338,9 @@ public class TelephonyManager {
* @return true if mobile data is enabled.
*/
@RequiresPermission(anyOf = {android.Manifest.permission.ACCESS_NETWORK_STATE,
- android.Manifest.permission.MODIFY_PHONE_STATE,
- android.Manifest.permission.READ_PHONE_STATE})
+ android.Manifest.permission.MODIFY_PHONE_STATE})
public boolean isDataEnabled() {
- return isDataEnabledForReason(DATA_ENABLED_REASON_USER);
+ return getDataEnabled(getSubId(SubscriptionManager.getDefaultDataSubscriptionId()));
}
/**
@@ -9586,7 +9585,7 @@ public class TelephonyManager {
@SystemApi
public boolean getDataEnabled(int subId) {
try {
- return isDataEnabledForReason(subId, DATA_ENABLED_REASON_USER);
+ return isDataEnabledForReason(DATA_ENABLED_REASON_USER);
} catch (RuntimeException e) {
Log.e(TAG, "Error calling isDataEnabledForReason e:" + e);
}