Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit a37d4448 authored by Sarah Chin's avatar Sarah Chin Committed by Automerger Merge Worker
Browse files

NetworkTypeController update on PCC changed for LTE/LTE+ switch am: b06cb6ab 

Original change: https://android-review.googlesource.com/c/platform/frameworks/opt/telephony/+/1733993

Change-Id: I1b57a40387d4aeec819e003d48e4d4cd6b7835c0
parents 7053c1aa b06cb6ab

src/java/com/android/internal/telephony/NetworkTypeController.java

+7 −8
Original line number Diff line number Diff line
@@ -608,15 +608,14 @@ public class NetworkTypeController extends StateMachine { 
                    // ignored

                    break;

                case EVENT_PHYSICAL_CHANNEL_CONFIG_CHANGED:

                    if (!mIsPhysicalChannelConfig16Supported) {

                        // ignore

                        break;

                    }

                    if (mIsPhysicalChannelConfig16Supported) {

                        mPhysicalLinkState = getPhysicalLinkStateFromPhysicalChannelConfig();

                        if (mIsTimerResetEnabledForLegacyStateRRCIdle && !isPhysicalLinkActive()) {

                            resetAllTimers();

                        updateOverrideNetworkType();

                        }

                    }

                    // Update in case of LTE/LTE+ switch

                    updateOverrideNetworkType();

                    break;

                case EVENT_PHYSICAL_LINK_STATE_CHANGED:

                    AsyncResult ar = (AsyncResult) msg.obj;

src/java/com/android/internal/telephony/ServiceStateTracker.java

+1 −1
Original line number Diff line number Diff line
@@ -1716,7 +1716,6 @@ public class ServiceStateTracker extends Handler { 
                        log("EVENT_PHYSICAL_CHANNEL_CONFIG: size=" + list.size() + " list="

                                + list);

                    }

                    mPhone.notifyPhysicalChannelConfig(list);

                    mLastPhysicalChannelConfigList = list;

                    boolean hasChanged = false;

                    if (updateNrStateFromPhysicalChannelConfigs(list, mSS)) {
@@ -1730,6 +1729,7 @@ public class ServiceStateTracker extends Handler { 
                    hasChanged |= RatRatcheter

                            .updateBandwidths(getBandwidthsFromConfigs(list), mSS);



                    mPhone.notifyPhysicalChannelConfig(list);

                    // Notify NR frequency, NR connection status or bandwidths changed.

                    if (hasChanged) {

                        mPhone.notifyServiceStateChanged(mSS);

tests/telephonytests/src/com/android/internal/telephony/NetworkTypeControllerTest.java

+17 −1
Original line number Diff line number Diff line
@@ -478,7 +478,6 @@ public class NetworkTypeControllerTest extends TelephonyTest { 
        mBundle.putInt(CarrierConfigManager.KEY_NR_ADVANCED_CAPABLE_PCO_ID_INT, 0xFF03);

        broadcastCarrierConfigs();





        mNetworkTypeController.sendMessage(EVENT_PCO_DATA_CHANGED,

                new AsyncResult(null, new PcoData(cid, "", 0xff03, contents), null));

        mNetworkTypeController.sendMessage(NetworkTypeController.EVENT_UPDATE);
@@ -565,6 +564,23 @@ public class NetworkTypeControllerTest extends TelephonyTest { 
        assertEquals("not_restricted_rrc_con", getCurrentState().getName());

    }



    @Test

    public void testPhysicalChannelChangeFromLteToLteCa() throws Exception {

        testTransitionToCurrentStateLegacy();

        doReturn(TelephonyManager.NETWORK_TYPE_LTE).when(mServiceState).getDataNetworkType();

        updateOverrideNetworkType();

        assertEquals(TelephonyDisplayInfo.OVERRIDE_NETWORK_TYPE_NONE,

                mNetworkTypeController.getOverrideNetworkType());



        doReturn(true).when(mServiceState).isUsingCarrierAggregation();

        doReturn(new int[] {30000}).when(mServiceState).getCellBandwidths();

        mNetworkTypeController.sendMessage(EVENT_PHYSICAL_CHANNEL_CONFIG_CHANGED);

        processAllMessages();



        assertEquals(TelephonyDisplayInfo.OVERRIDE_NETWORK_TYPE_LTE_CA,

                mNetworkTypeController.getOverrideNetworkType());

    }



    @Test

    public void testEventPhysicalLinkStateChanged() throws Exception {

        testTransitionToCurrentStateLteConnected();