From 3b3704d693caf720ef0cc258750f8dacedd10583 Mon Sep 17 00:00:00 2001 From: Sarah Chin Date: Fri, 6 Nov 2020 11:30:45 -0800 Subject: [PATCH] Update signal strength on ServiceState in service Test: make RunSettingsRobotTests Bug: 167643728 Change-Id: Ibf69f9e9ec2acafef1316069d37b8d2a27d7d4d7 --- .../deviceinfo/simstatus/SimStatusDialogController.java | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/com/android/settings/deviceinfo/simstatus/SimStatusDialogController.java b/src/com/android/settings/deviceinfo/simstatus/SimStatusDialogController.java index 9858ad21026..e3f217dba3a 100644 --- a/src/com/android/settings/deviceinfo/simstatus/SimStatusDialogController.java +++ b/src/com/android/settings/deviceinfo/simstatus/SimStatusDialogController.java @@ -140,6 +140,7 @@ public class SimStatusDialogController implements LifecycleObserver, OnResume, O private SubscriptionInfo mSubscriptionInfo; private TelephonyDisplayInfo mTelephonyDisplayInfo; + private ServiceState mServiceState; private final int mSlotIndex; private TelephonyManager mTelephonyManager; @@ -437,6 +438,9 @@ public class SimStatusDialogController implements LifecycleObserver, OnResume, O final int state = Utils.getCombinedServiceState(serviceState); if (!Utils.isInService(serviceState)) { resetSignalStrength(); + } else if (mServiceState != null && !Utils.isInService(mServiceState)) { + // If ServiceState changed from out of service -> in service, update signal strength. + updateSignalStrength(mTelephonyManager.getSignalStrength()); } String serviceStateValue; @@ -484,7 +488,7 @@ public class SimStatusDialogController implements LifecycleObserver, OnResume, O } ServiceState serviceState = mTelephonyManager.getServiceState(); - if (serviceState == null || !Utils.isInService(serviceState)) { + if (!Utils.isInService(serviceState)) { return; } @@ -767,6 +771,7 @@ public class SimStatusDialogController implements LifecycleObserver, OnResume, O updateNetworkProvider(); updateServiceState(serviceState); updateRoamingStatus(serviceState); + mServiceState = serviceState; } @Override