Merge "When data switches during voice call, make sure signal bar updates it."

am: b6ae55b169

Change-Id: I74484d5f2cecd5fd5c3b5e5592fdd90778bcf925
This commit is contained in:
Xiangyu/Malcolm Chen
2019-06-25 11:19:23 -07:00
committed by android-build-merger
3 changed files with 7 additions and 6 deletions

View File

@@ -548,7 +548,7 @@ public class MobileSignalController extends SignalController<
} }
private boolean isDataDisabled() { private boolean isDataDisabled() {
return !mPhone.getDataEnabled(mSubscriptionInfo.getSubscriptionId()); return !mPhone.isDataCapable();
} }
@VisibleForTesting @VisibleForTesting
@@ -568,6 +568,7 @@ public class MobileSignalController extends SignalController<
pw.println(" mSignalStrength=" + mSignalStrength + ","); pw.println(" mSignalStrength=" + mSignalStrength + ",");
pw.println(" mDataState=" + mDataState + ","); pw.println(" mDataState=" + mDataState + ",");
pw.println(" mDataNetType=" + mDataNetType + ","); pw.println(" mDataNetType=" + mDataNetType + ",");
pw.println(" isDataDisabled=" + isDataDisabled() + ",");
} }
class MobilePhoneStateListener extends PhoneStateListener { class MobilePhoneStateListener extends PhoneStateListener {

View File

@@ -155,7 +155,7 @@ public class NetworkControllerBaseTest extends SysuiTestCase {
protected void setupNetworkController() { protected void setupNetworkController() {
// For now just pretend to be the data sim, so we can test that too. // For now just pretend to be the data sim, so we can test that too.
mSubId = SubscriptionManager.DEFAULT_SUBSCRIPTION_ID; mSubId = SubscriptionManager.DEFAULT_SUBSCRIPTION_ID;
when(mMockTm.getDataEnabled(mSubId)).thenReturn(true); when(mMockTm.isDataCapable()).thenReturn(true);
setDefaultSubId(mSubId); setDefaultSubId(mSubId);
setSubscriptions(mSubId); setSubscriptions(mSubId);
mMobileSignalController = mNetworkController.mMobileSignalControllers.get(mSubId); mMobileSignalController = mNetworkController.mMobileSignalControllers.get(mSubId);

View File

@@ -119,7 +119,7 @@ public class NetworkControllerDataTest extends NetworkControllerBaseTest {
@Test @Test
public void testNoInternetIcon() { public void testNoInternetIcon() {
setupNetworkController(); setupNetworkController();
when(mMockTm.getDataEnabled(mSubId)).thenReturn(false); when(mMockTm.isDataCapable()).thenReturn(false);
setupDefaultSignal(); setupDefaultSignal();
updateDataConnectionState(TelephonyManager.DATA_CONNECTED, 0); updateDataConnectionState(TelephonyManager.DATA_CONNECTED, 0);
setConnectivityViaBroadcast(NetworkCapabilities.TRANSPORT_CELLULAR, false, false); setConnectivityViaBroadcast(NetworkCapabilities.TRANSPORT_CELLULAR, false, false);
@@ -133,7 +133,7 @@ public class NetworkControllerDataTest extends NetworkControllerBaseTest {
@Test @Test
public void testDataDisabledIcon() { public void testDataDisabledIcon() {
setupNetworkController(); setupNetworkController();
when(mMockTm.getDataEnabled(mSubId)).thenReturn(false); when(mMockTm.isDataCapable()).thenReturn(false);
setupDefaultSignal(); setupDefaultSignal();
updateDataConnectionState(TelephonyManager.DATA_DISCONNECTED, 0); updateDataConnectionState(TelephonyManager.DATA_DISCONNECTED, 0);
setConnectivityViaBroadcast(NetworkCapabilities.TRANSPORT_CELLULAR, false, false); setConnectivityViaBroadcast(NetworkCapabilities.TRANSPORT_CELLULAR, false, false);
@@ -188,7 +188,7 @@ public class NetworkControllerDataTest extends NetworkControllerBaseTest {
@Test @Test
public void testDataDisabledIcon_UserNotSetup() { public void testDataDisabledIcon_UserNotSetup() {
setupNetworkController(); setupNetworkController();
when(mMockTm.getDataEnabled(mSubId)).thenReturn(false); when(mMockTm.isDataCapable()).thenReturn(false);
setupDefaultSignal(); setupDefaultSignal();
updateDataConnectionState(TelephonyManager.DATA_DISCONNECTED, 0); updateDataConnectionState(TelephonyManager.DATA_DISCONNECTED, 0);
setConnectivityViaBroadcast(NetworkCapabilities.TRANSPORT_CELLULAR, false, false); setConnectivityViaBroadcast(NetworkCapabilities.TRANSPORT_CELLULAR, false, false);
@@ -203,7 +203,7 @@ public class NetworkControllerDataTest extends NetworkControllerBaseTest {
@Test @Test
public void testAlwaysShowDataRatIcon() { public void testAlwaysShowDataRatIcon() {
setupDefaultSignal(); setupDefaultSignal();
when(mMockTm.getDataEnabled(mSubId)).thenReturn(false); when(mMockTm.isDataCapable()).thenReturn(false);
updateDataConnectionState(TelephonyManager.DATA_DISCONNECTED, updateDataConnectionState(TelephonyManager.DATA_DISCONNECTED,
TelephonyManager.NETWORK_TYPE_GSM); TelephonyManager.NETWORK_TYPE_GSM);