diff options
author | TreeHugger Robot <treehugger-gerrit@google.com> | 2019-05-15 23:02:11 +0000 |
---|---|---|
committer | Android (Google) Code Review <android-gerrit@google.com> | 2019-05-15 23:02:11 +0000 |
commit | eea7008c22a044d0c7339cca42e5645d2b30c680 (patch) | |
tree | 3da34898767d92edf89bca41a059204ecf07bc01 | |
parent | d148f6630db788d65140793f1328dafb54c57a1b (diff) | |
parent | 41aaf0059dfc771a509f1bd0d004e8bee7497617 (diff) | |
download | telephony-temp_b_132622481_stage-aosp-master.tar.gz |
Merge "DO NOT MERGE - Merge pi-platform-release (PPRL.190505.001) into stage-aosp-master" into stage-aosp-mastertemp_b_132622481_stage-aosp-master
-rw-r--r-- | src/java/com/android/internal/telephony/GsmCdmaPhone.java | 7 | ||||
-rw-r--r-- | tests/telephonytests/src/com/android/internal/telephony/GsmCdmaPhoneTest.java | 26 |
2 files changed, 33 insertions, 0 deletions
diff --git a/src/java/com/android/internal/telephony/GsmCdmaPhone.java b/src/java/com/android/internal/telephony/GsmCdmaPhone.java index fa5ca8e84a..fb6a8c79ed 100644 --- a/src/java/com/android/internal/telephony/GsmCdmaPhone.java +++ b/src/java/com/android/internal/telephony/GsmCdmaPhone.java @@ -1757,6 +1757,13 @@ public class GsmCdmaPhone extends Phone { IccRecords r = mIccRecords.get(); return (r != null) ? r.getMsisdnNumber() : null; } else { + CarrierConfigManager configManager = (CarrierConfigManager) + mContext.getSystemService(Context.CARRIER_CONFIG_SERVICE); + boolean use_usim = configManager.getConfigForSubId(getSubId()).getBoolean( + CarrierConfigManager.KEY_USE_USIM_BOOL); + if (use_usim) { + return (mSimRecords != null) ? mSimRecords.getMsisdnNumber() : null; + } return mSST.getMdnNumber(); } } diff --git a/tests/telephonytests/src/com/android/internal/telephony/GsmCdmaPhoneTest.java b/tests/telephonytests/src/com/android/internal/telephony/GsmCdmaPhoneTest.java index fe614f0caa..8655279068 100644 --- a/tests/telephonytests/src/com/android/internal/telephony/GsmCdmaPhoneTest.java +++ b/tests/telephonytests/src/com/android/internal/telephony/GsmCdmaPhoneTest.java @@ -975,4 +975,30 @@ public class GsmCdmaPhoneTest extends TelephonyTest { ss.addNetworkRegistrationInfo(nri); assertEquals(mPhoneUT.getCsCallRadioTech(), ServiceState.RIL_RADIO_TECHNOLOGY_UNKNOWN); } + + @Test + @SmallTest + public void testGetLine1NumberForGsmPhone() { + final String msisdn = "+1234567890"; + doReturn(msisdn).when(mSimRecords).getMsisdnNumber(); + + switchToGsm(); + assertEquals(msisdn, mPhoneUT.getLine1Number()); + } + + @Test + @SmallTest + public void testGetLine1NumberForCdmaPhone() { + final String mdn = "1234567890"; + final String msisdn = "+1234567890"; + doReturn(mdn).when(mSST).getMdnNumber(); + doReturn(msisdn).when(mSimRecords).getMsisdnNumber(); + + switchToCdma(); + assertEquals(mdn, mPhoneUT.getLine1Number()); + + mContextFixture.getCarrierConfigBundle().putBoolean( + CarrierConfigManager.KEY_USE_USIM_BOOL, true); + assertEquals(msisdn, mPhoneUT.getLine1Number()); + } } |