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

Commit 88cb8282 authored by Amit Mahajan's avatar Amit Mahajan Committed by Automerger Merge Worker
Browse files

Delete unused intents. am: 7ae34522 

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



Change-Id: I5d48b3382fcdf87cd5defabf86f2664852d07980
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 8f91778c 7ae34522

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

+1 −1
Original line number Diff line number Diff line
@@ -193,7 +193,7 @@ public class CarrierResolver extends Handler { 
    /**

     * This is triggered from SubscriptionInfoUpdater after sim state change.

     * The sequence of sim loading would be

     *  1. ACTION_SUBINFO_CONTENT_CHANGE

     *  1. OnSubscriptionsChangedListener

     *  2. ACTION_SIM_STATE_CHANGED/ACTION_SIM_CARD_STATE_CHANGED

     *  /ACTION_SIM_APPLICATION_STATE_CHANGED

     *  3. ACTION_SUBSCRIPTION_CARRIER_IDENTITY_CHANGED

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

+0 −14
Original line number Diff line number Diff line
@@ -496,17 +496,6 @@ public class SubscriptionController extends ISub.Stub { 
        }

    }



    /**

     * Broadcast when SubscriptionInfo has changed

     * FIXME: Hopefully removed if the API council accepts SubscriptionInfoListener

     */

     private void broadcastSimInfoContentChanged() {

        Intent intent = new Intent(TelephonyIntents.ACTION_SUBINFO_CONTENT_CHANGE);

        mContext.sendBroadcast(intent);

        intent = new Intent(TelephonyIntents.ACTION_SUBINFO_RECORD_UPDATED);

        mContext.sendBroadcast(intent);

     }



    /**

     * Notify the changed of subscription info.

     */
@@ -518,9 +507,6 @@ public class SubscriptionController extends ISub.Stub { 
        if (DBG) logd("notifySubscriptionInfoChanged:");

        trm.notifySubscriptionInfoChanged();



        // FIXME: Remove if listener technique accepted.

        broadcastSimInfoContentChanged();



        MultiSimSettingController.getInstance().notifySubscriptionInfoChanged();

        TelephonyMetrics metrics = TelephonyMetrics.getInstance();

        List<SubscriptionInfo> subInfos;

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

+2 −1
Original line number Diff line number Diff line
@@ -612,7 +612,8 @@ public class SubscriptionInfoUpdater extends Handler { 


        /**

         * The sim loading sequence will be

         *  1. ACTION_SUBINFO_CONTENT_CHANGE happens through updateSubscriptionInfoByIccId() above.

         *  1. OnSubscriptionsChangedListener is called through updateSubscriptionInfoByIccId()

         *  above.

         *  2. ACTION_SIM_STATE_CHANGED/ACTION_SIM_CARD_STATE_CHANGED

         *  /ACTION_SIM_APPLICATION_STATE_CHANGED

         *  3. ACTION_SUBSCRIPTION_CARRIER_IDENTITY_CHANGED

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

+0 −25
Original line number Diff line number Diff line
@@ -284,12 +284,6 @@ public class SubscriptionControllerTest extends TelephonyTest { 
        assertEquals(iconTint, subInfo.getIconTint());

        assertEquals(disNum, subInfo.getNumber());

        assertEquals(isOpportunistic, subInfo.isOpportunistic());



        /* verify broadcast intent */

        ArgumentCaptor<Intent> captorIntent = ArgumentCaptor.forClass(Intent.class);

        verify(mContext, atLeast(1)).sendBroadcast(captorIntent.capture());

        assertEquals(TelephonyIntents.ACTION_SUBINFO_RECORD_UPDATED,

                captorIntent.getValue().getAction());

    }



    @Test @SmallTest
@@ -310,13 +304,6 @@ public class SubscriptionControllerTest extends TelephonyTest { 
        assertNotNull(subInfo);

        assertEquals(disName, subInfo.getDisplayName());

        assertEquals(nameSource, subInfo.getNameSource());



        /* verify broadcast intent */

        ArgumentCaptor<Intent> captorIntent = ArgumentCaptor.forClass(Intent.class);

        verify(mContext, atLeast(1)).sendBroadcast(captorIntent.capture());

        assertEquals(TelephonyIntents.ACTION_SUBINFO_RECORD_UPDATED,

                captorIntent.getValue().getAction());



    }



    private void setSimEmbedded(boolean isEmbedded) throws Exception {
@@ -595,12 +582,6 @@ public class SubscriptionControllerTest extends TelephonyTest { 
        assertNotNull(subInfo);

        assertEquals(Integer.parseInt(mCcMncVERIZON.substring(0, 3)), subInfo.getMcc());

        assertEquals(Integer.parseInt(mCcMncVERIZON.substring(3)), subInfo.getMnc());



         /* verify broadcast intent */

        ArgumentCaptor<Intent> captorIntent = ArgumentCaptor.forClass(Intent.class);

        verify(mContext, atLeast(1)).sendBroadcast(captorIntent.capture());

        assertEquals(TelephonyIntents.ACTION_SUBINFO_RECORD_UPDATED,

                captorIntent.getValue().getAction());

    }



    @Test @SmallTest
@@ -613,12 +594,6 @@ public class SubscriptionControllerTest extends TelephonyTest { 
                .getActiveSubscriptionInfo(1, mCallingPackage, mCallingFeature);

        assertNotNull(subInfo);

        assertEquals(carrierId, subInfo.getCarrierId());



         /* verify broadcast intent */

        ArgumentCaptor<Intent> captorIntent = ArgumentCaptor.forClass(Intent.class);

        verify(mContext, atLeast(1)).sendBroadcast(captorIntent.capture());

        assertEquals(TelephonyIntents.ACTION_SUBINFO_RECORD_UPDATED,

                captorIntent.getValue().getAction());

    }



    @Test