diff options
| author | 2021-05-24 12:57:49 -0700 | |
|---|---|---|
| committer | 2021-05-24 16:16:50 -0700 | |
| commit | 0bacf4f46bc2ba465f81c352f394c12238e8ed14 (patch) | |
| tree | a29f2911e814a5ea16b676daa179bb654477ee43 | |
| parent | 94a4cdd4e81e276d9c35b5b9e3f8353bc2b11590 (diff) | |
Update javadoc of TM#getDataNetworkType to align with implementation
Javadoc of TelephonyManager#getDataNetworkType says if the
TelephonyManager object is not created with createForSubscriptionId,
then applies to SubscriptionManager#getDefaultDataSubscriptionId().
But the actual implementation applies to
SubscriptionManager#getActiveDataSubscriptionId() instead. While they
are not always the same (e.g. when opptunistic network is active), the
implementation is more accurate than the javadoc.
Bug: 144803512
Test: make docs
Change-Id: I7db32a9a99dd5d7f7c32b6b33f3d5f7bd09a0e02
| -rw-r--r-- | telephony/java/android/telephony/TelephonyManager.java | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/telephony/java/android/telephony/TelephonyManager.java b/telephony/java/android/telephony/TelephonyManager.java index 1860ecb7c274..26cd568d6530 100644 --- a/telephony/java/android/telephony/TelephonyManager.java +++ b/telephony/java/android/telephony/TelephonyManager.java @@ -2932,7 +2932,12 @@ public class TelephonyManager { * currently in use on the device for data transmission. * * If this object has been created with {@link #createForSubscriptionId}, applies to the given - * subId. Otherwise, applies to {@link SubscriptionManager#getDefaultDataSubscriptionId()} + * subId. Otherwise, applies to {@link SubscriptionManager#getActiveDataSubscriptionId()}. + * + * Note: Before {@link SubscriptionManager#getActiveDataSubscriptionId()} was introduced in API + * level 30, it was applied to {@link SubscriptionManager#getDefaultDataSubscriptionId()} which + * may be different now from {@link SubscriptionManager#getActiveDataSubscriptionId()}, e.g. + * when opportunistic network is providing cellular internet connection to the user. * * <p>Requires Permission: {@link android.Manifest.permission#READ_PHONE_STATE READ_PHONE_STATE} * or that the calling app has carrier privileges (see {@link #hasCarrierPrivileges}). |