Merge "Rename references to getLteOnCdmaMode"

This commit is contained in:
Sarah Chin
2020-02-21 22:09:16 +00:00
committed by Gerrit Code Review
6 changed files with 12 additions and 34 deletions

View File

@@ -60,7 +60,6 @@ import android.widget.Button;
import android.widget.ProgressBar; import android.widget.ProgressBar;
import android.widget.TextView; import android.widget.TextView;
import com.android.internal.telephony.PhoneConstants;
import com.android.internal.widget.LockPatternUtils; import com.android.internal.widget.LockPatternUtils;
import com.android.internal.widget.LockPatternView; import com.android.internal.widget.LockPatternView;
import com.android.internal.widget.LockPatternView.Cell; import com.android.internal.widget.LockPatternView.Cell;
@@ -916,9 +915,7 @@ public class CryptKeeper extends Activity implements TextView.OnEditorActionList
* phone that has no encryption. * phone that has no encryption.
*/ */
private final void setAirplaneModeIfNecessary() { private final void setAirplaneModeIfNecessary() {
final boolean isLteDevice = if (!getTelephonyManager().isGlobalModeEnabled()) {
getTelephonyManager().getLteOnCdmaMode() == PhoneConstants.LTE_ON_CDMA_TRUE;
if (!isLteDevice) {
Log.d(TAG, "Going into airplane mode."); Log.d(TAG, "Going into airplane mode.");
Settings.Global.putInt(getContentResolver(), Settings.Global.AIRPLANE_MODE_ON, 1); Settings.Global.putInt(getContentResolver(), Settings.Global.AIRPLANE_MODE_ON, 1);
final Intent intent = new Intent(Intent.ACTION_AIRPLANE_MODE_CHANGED); final Intent intent = new Intent(Intent.ACTION_AIRPLANE_MODE_CHANGED);

View File

@@ -31,7 +31,6 @@ import android.util.Log;
import androidx.annotation.NonNull; import androidx.annotation.NonNull;
import androidx.annotation.VisibleForTesting; import androidx.annotation.VisibleForTesting;
import com.android.internal.telephony.PhoneConstants;
import com.android.settings.R; import com.android.settings.R;
public class ImeiInfoDialogController { public class ImeiInfoDialogController {
@@ -147,8 +146,7 @@ public class ImeiInfoDialogController {
@VisibleForTesting @VisibleForTesting
boolean isCdmaLteEnabled() { boolean isCdmaLteEnabled() {
return mTelephonyManager.getLteOnCdmaMode(mSubscriptionInfo.getSubscriptionId()) return mTelephonyManager.isGlobalModeEnabled();
== PhoneConstants.LTE_ON_CDMA_TRUE;
} }
@VisibleForTesting @VisibleForTesting

View File

@@ -28,8 +28,6 @@ import android.text.TextUtils;
import androidx.preference.Preference; import androidx.preference.Preference;
import com.android.internal.telephony.PhoneConstants;
/** /**
* Preference controller for "Data service setup" * Preference controller for "Data service setup"
*/ */
@@ -49,14 +47,12 @@ public class DataServiceSetupPreferenceController extends TelephonyBasePreferenc
@Override @Override
public int getAvailabilityStatus(int subId) { public int getAvailabilityStatus(int subId) {
final boolean isLteOnCdma = mTelephonyManager.getLteOnCdmaMode()
== PhoneConstants.LTE_ON_CDMA_TRUE;
final PersistableBundle carrierConfig = mCarrierConfigManager.getConfigForSubId(subId); final PersistableBundle carrierConfig = mCarrierConfigManager.getConfigForSubId(subId);
return subId != SubscriptionManager.INVALID_SUBSCRIPTION_ID return subId != SubscriptionManager.INVALID_SUBSCRIPTION_ID
&& carrierConfig != null && carrierConfig != null
&& !carrierConfig.getBoolean( && !carrierConfig.getBoolean(
CarrierConfigManager.KEY_HIDE_CARRIER_NETWORK_SETTINGS_BOOL) CarrierConfigManager.KEY_HIDE_CARRIER_NETWORK_SETTINGS_BOOL)
&& isLteOnCdma && !TextUtils.isEmpty(mSetupUrl) && mTelephonyManager.isGlobalModeEnabled() && !TextUtils.isEmpty(mSetupUrl)
? AVAILABLE ? AVAILABLE
: CONDITIONALLY_UNAVAILABLE; : CONDITIONALLY_UNAVAILABLE;
} }

View File

@@ -151,9 +151,7 @@ public class EnabledNetworkModePreferenceController extends
mTelephonyManager = mContext.getSystemService(TelephonyManager.class) mTelephonyManager = mContext.getSystemService(TelephonyManager.class)
.createForSubscriptionId(mSubId); .createForSubscriptionId(mSubId);
final boolean isLteOnCdma = mIsGlobalCdma = mTelephonyManager.isGlobalModeEnabled()
mTelephonyManager.getLteOnCdmaMode() == PhoneConstants.LTE_ON_CDMA_TRUE;
mIsGlobalCdma = isLteOnCdma
&& carrierConfig.getBoolean(CarrierConfigManager.KEY_SHOW_CDMA_CHOICES_BOOL); && carrierConfig.getBoolean(CarrierConfigManager.KEY_SHOW_CDMA_CHOICES_BOOL);
mShow4GForLTE = carrierConfig != null mShow4GForLTE = carrierConfig != null
? carrierConfig.getBoolean( ? carrierConfig.getBoolean(
@@ -181,13 +179,11 @@ public class EnabledNetworkModePreferenceController extends
mContext.getContentResolver(), mContext.getContentResolver(),
android.provider.Settings.Global.LTE_SERVICE_FORCED + mSubId, android.provider.Settings.Global.LTE_SERVICE_FORCED + mSubId,
0); 0);
final boolean isLteOnCdma = mTelephonyManager.getLteOnCdmaMode()
== PhoneConstants.LTE_ON_CDMA_TRUE;
final int settingsNetworkMode = android.provider.Settings.Global.getInt( final int settingsNetworkMode = android.provider.Settings.Global.getInt(
mContext.getContentResolver(), mContext.getContentResolver(),
android.provider.Settings.Global.PREFERRED_NETWORK_MODE + mSubId, android.provider.Settings.Global.PREFERRED_NETWORK_MODE + mSubId,
Phone.PREFERRED_NT_MODE); Phone.PREFERRED_NT_MODE);
if (isLteOnCdma) { if (mTelephonyManager.isGlobalModeEnabled()) {
if (lteForced == 0) { if (lteForced == 0) {
preference.setEntries( preference.setEntries(
R.array.enabled_networks_cdma_choices); R.array.enabled_networks_cdma_choices);

View File

@@ -100,9 +100,7 @@ public class PreferredNetworkModePreferenceController extends TelephonyBasePrefe
mTelephonyManager = mContext.getSystemService(TelephonyManager.class) mTelephonyManager = mContext.getSystemService(TelephonyManager.class)
.createForSubscriptionId(mSubId); .createForSubscriptionId(mSubId);
final boolean isLteOnCdma = mIsGlobalCdma = mTelephonyManager.isGlobalModeEnabled()
mTelephonyManager.getLteOnCdmaMode() == PhoneConstants.LTE_ON_CDMA_TRUE;
mIsGlobalCdma = isLteOnCdma
&& carrierConfig.getBoolean(CarrierConfigManager.KEY_SHOW_CDMA_CHOICES_BOOL); && carrierConfig.getBoolean(CarrierConfigManager.KEY_SHOW_CDMA_CHOICES_BOOL);
} }
@@ -129,12 +127,9 @@ public class PreferredNetworkModePreferenceController extends TelephonyBasePrefe
case TelephonyManagerConstants.NETWORK_MODE_GSM_UMTS: case TelephonyManagerConstants.NETWORK_MODE_GSM_UMTS:
return R.string.preferred_network_mode_gsm_wcdma_summary; return R.string.preferred_network_mode_gsm_wcdma_summary;
case TelephonyManagerConstants.NETWORK_MODE_CDMA_EVDO: case TelephonyManagerConstants.NETWORK_MODE_CDMA_EVDO:
switch (mTelephonyManager.getLteOnCdmaMode()) { return mTelephonyManager.isGlobalModeEnabled()
case PhoneConstants.LTE_ON_CDMA_TRUE: ? R.string.preferred_network_mode_cdma_summary
return R.string.preferred_network_mode_cdma_summary; : R.string.preferred_network_mode_cdma_evdo_summary;
default:
return R.string.preferred_network_mode_cdma_evdo_summary;
}
case TelephonyManagerConstants.NETWORK_MODE_CDMA_NO_EVDO: case TelephonyManagerConstants.NETWORK_MODE_CDMA_NO_EVDO:
return R.string.preferred_network_mode_cdma_only_summary; return R.string.preferred_network_mode_cdma_only_summary;
case TelephonyManagerConstants.NETWORK_MODE_EVDO_NO_CDMA: case TelephonyManagerConstants.NETWORK_MODE_EVDO_NO_CDMA:

View File

@@ -24,9 +24,7 @@ import static com.google.common.truth.Truth.assertThat;
import static org.mockito.Mockito.doNothing; import static org.mockito.Mockito.doNothing;
import static org.mockito.Mockito.doReturn; import static org.mockito.Mockito.doReturn;
import static org.mockito.Mockito.spy; import static org.mockito.Mockito.spy;
import static org.mockito.Mockito.verify;
import android.app.Activity;
import android.content.Context; import android.content.Context;
import android.content.Intent; import android.content.Intent;
import android.net.Uri; import android.net.Uri;
@@ -39,7 +37,6 @@ import android.text.TextUtils;
import androidx.preference.Preference; import androidx.preference.Preference;
import com.android.internal.telephony.PhoneConstants;
import com.android.settingslib.RestrictedPreference; import com.android.settingslib.RestrictedPreference;
import org.junit.Before; import org.junit.Before;
@@ -48,7 +45,6 @@ import org.junit.runner.RunWith;
import org.mockito.ArgumentCaptor; import org.mockito.ArgumentCaptor;
import org.mockito.Mock; import org.mockito.Mock;
import org.mockito.MockitoAnnotations; import org.mockito.MockitoAnnotations;
import org.robolectric.Robolectric;
import org.robolectric.RobolectricTestRunner; import org.robolectric.RobolectricTestRunner;
import org.robolectric.RuntimeEnvironment; import org.robolectric.RuntimeEnvironment;
@@ -94,7 +90,7 @@ public class DataServiceSetupPreferenceControllerTest {
@Test @Test
public void getAvailabilityStatus_allConfigOn_returnAvailable() { public void getAvailabilityStatus_allConfigOn_returnAvailable() {
doReturn(PhoneConstants.LTE_ON_CDMA_TRUE).when(mTelephonyManager).getLteOnCdmaMode(); doReturn(true).when(mTelephonyManager).isGlobalModeEnabled();
mCarrierConfig.putBoolean(CarrierConfigManager.KEY_HIDE_CARRIER_NETWORK_SETTINGS_BOOL, mCarrierConfig.putBoolean(CarrierConfigManager.KEY_HIDE_CARRIER_NETWORK_SETTINGS_BOOL,
false); false);
@@ -105,7 +101,7 @@ public class DataServiceSetupPreferenceControllerTest {
public void getAvailabilityStatus_missUrl_returnUnavailable() { public void getAvailabilityStatus_missUrl_returnUnavailable() {
Settings.Global.putString(mContext.getContentResolver(), Settings.Global.putString(mContext.getContentResolver(),
Settings.Global.SETUP_PREPAID_DATA_SERVICE_URL, ""); Settings.Global.SETUP_PREPAID_DATA_SERVICE_URL, "");
doReturn(PhoneConstants.LTE_ON_CDMA_TRUE).when(mTelephonyManager).getLteOnCdmaMode(); doReturn(true).when(mTelephonyManager).isGlobalModeEnabled();
mCarrierConfig.putBoolean(CarrierConfigManager.KEY_HIDE_CARRIER_NETWORK_SETTINGS_BOOL, mCarrierConfig.putBoolean(CarrierConfigManager.KEY_HIDE_CARRIER_NETWORK_SETTINGS_BOOL,
false); false);
@@ -117,7 +113,7 @@ public class DataServiceSetupPreferenceControllerTest {
@Test @Test
public void getAvailabilityStatus_notCdma_returnUnavailable() { public void getAvailabilityStatus_notCdma_returnUnavailable() {
doReturn(PhoneConstants.LTE_ON_CDMA_FALSE).when(mTelephonyManager).getLteOnCdmaMode(); doReturn(false).when(mTelephonyManager).isGlobalModeEnabled();
mCarrierConfig.putBoolean(CarrierConfigManager.KEY_HIDE_CARRIER_NETWORK_SETTINGS_BOOL, mCarrierConfig.putBoolean(CarrierConfigManager.KEY_HIDE_CARRIER_NETWORK_SETTINGS_BOOL,
false); false);