diff options
author | Arne Coucheron <arco68@gmail.com> | 2016-12-02 01:29:47 +0100 |
---|---|---|
committer | Michael Bestas <mikeioannina@gmail.com> | 2017-01-07 01:41:45 +0200 |
commit | 3c44ab0ad205c791ec8bbba9451616fe629f5b50 (patch) | |
tree | 71708c097122881925d91edefb929902af8ebd06 | |
parent | 663cbef592e551b11ef9db772f1c181391e79cc2 (diff) | |
download | android_frameworks_opt_telephony-3c44ab0ad205c791ec8bbba9451616fe629f5b50.tar.gz android_frameworks_opt_telephony-3c44ab0ad205c791ec8bbba9451616fe629f5b50.tar.bz2 android_frameworks_opt_telephony-3c44ab0ad205c791ec8bbba9451616fe629f5b50.zip |
Fixes and cleanups after 7.1.0/7.1.1 merge
Fix some merge derps and sync code properly to upstream.
Also remove stray file that sneaked in after 7.1.1 merge.
Change-Id: Ifc93a58016e4e8b4f4d83f72281b306823474ef1
3 files changed, 2 insertions, 12 deletions
diff --git a/src/java/com/android/internal/telephony/GsmCdmaConnection.java b/src/java/com/android/internal/telephony/GsmCdmaConnection.java index 9e298000f..28c94c841 100644 --- a/src/java/com/android/internal/telephony/GsmCdmaConnection.java +++ b/src/java/com/android/internal/telephony/GsmCdmaConnection.java @@ -165,7 +165,8 @@ public class GsmCdmaConnection extends Connection { } if (isPhoneTypeGsm()) { mDialString = dialString; - } else { + } + if (!isPhoneTypeGsm()) { Rlog.d(LOG_TAG, "[GsmCdmaConn] GsmCdmaConnection: dialString=" + maskDialString(dialString)); if (showOrigDialString) { diff --git a/src/java/com/android/internal/telephony/cdma/CdmaSMSDispatcher.java b/src/java/com/android/internal/telephony/cdma/CdmaSMSDispatcher.java index d1c91a159..5582cf959 100644 --- a/src/java/com/android/internal/telephony/cdma/CdmaSMSDispatcher.java +++ b/src/java/com/android/internal/telephony/cdma/CdmaSMSDispatcher.java @@ -27,13 +27,11 @@ import android.provider.Telephony.Sms; import android.telephony.Rlog; import android.telephony.ServiceState; import android.telephony.SmsManager; -import android.telephony.TelephonyManager; import com.android.internal.telephony.GsmAlphabet; import com.android.internal.telephony.GsmCdmaPhone; import com.android.internal.telephony.ImsSMSDispatcher; import com.android.internal.telephony.Phone; -import com.android.internal.telephony.PhoneConstants; import com.android.internal.telephony.SMSDispatcher; import com.android.internal.telephony.SmsConstants; import com.android.internal.telephony.SmsHeader; @@ -257,14 +255,6 @@ public class CdmaSMSDispatcher extends SMSDispatcher { Message reply = obtainMessage(EVENT_SEND_SMS_COMPLETE, tracker); byte[] pdu = (byte[]) tracker.getData().get("pdu"); - int currentDataNetwork = mPhone.getServiceState().getDataNetworkType(); - boolean imsSmsDisabled = (currentDataNetwork == TelephonyManager.NETWORK_TYPE_EHRPD - || (ServiceState.isLte(currentDataNetwork) - && !mPhone.getServiceStateTracker().isConcurrentVoiceAndDataAllowed())) - && mPhone.getServiceState().getVoiceNetworkType() - == TelephonyManager.NETWORK_TYPE_1xRTT - && ((GsmCdmaPhone) mPhone).mCT.mState != PhoneConstants.State.IDLE; - // sms over cdma is used: // if sms over IMS is not supported AND // this is not a retry case after sms over IMS failed diff --git a/tests/telephonytests/src/com/android/internal/telephony/gsm/GsmInboundSmsHandlerTest.java b/tests/telephonytests/src/com/android/internal/telephony/gsm/GsmInboundSmsHandlerTest.java index c0a958243..41415ecc2 100644 --- a/tests/telephonytests/src/com/android/internal/telephony/gsm/GsmInboundSmsHandlerTest.java +++ b/tests/telephonytests/src/com/android/internal/telephony/gsm/GsmInboundSmsHandlerTest.java @@ -32,7 +32,6 @@ import android.os.UserManager; import android.platform.test.annotations.Postsubmit; import android.provider.Telephony; import android.test.AssertionFailedError; -import android.test.mock.MockContentProvider; import android.test.mock.MockContentResolver; import android.test.suitebuilder.annotation.MediumTest; |