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

Commit 30782d10 authored by chen xu's avatar chen xu
Browse files

do downloading on handler thread to avoid ANR 

CarrierKeyDownloader should do lengthy operation like
downloading and write files in the handler thread rather than
main thread to avoid ANR.

Bug: 131715201
Test: Manual
Change-Id: I2959e54b712a3dc8d672072549b1ea124cd64800
parent 4a3ddadc

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

+25 −8
Original line number Original line Diff line number Diff line
@@ -31,6 +31,8 @@ import android.content.IntentFilter; 
import android.content.SharedPreferences;

import android.content.SharedPreferences;

import android.database.Cursor;

import android.database.Cursor;

import android.net.Uri;

import android.net.Uri;

import android.os.Handler;

import android.os.Message;

import android.os.PersistableBundle;

import android.os.PersistableBundle;

import android.telephony.CarrierConfigManager;

import android.telephony.CarrierConfigManager;

import android.telephony.ImsiEncryptionInfo;

import android.telephony.ImsiEncryptionInfo;
@@ -65,7 +67,7 @@ import java.util.zip.GZIPInputStream; 
 * This class contains logic to get Certificates and keep them current.

 * This class contains logic to get Certificates and keep them current.

 * The class will be instantiated by various Phone implementations.

 * The class will be instantiated by various Phone implementations.

 */

 */

public class CarrierKeyDownloadManager {

public class CarrierKeyDownloadManager extends Handler {

    private static final String LOG_TAG = "CarrierKeyDownloadManager";

    private static final String LOG_TAG = "CarrierKeyDownloadManager";





    private static final String MCC_MNC_PREF_TAG = "CARRIER_KEY_DM_MCC_MNC";

    private static final String MCC_MNC_PREF_TAG = "CARRIER_KEY_DM_MCC_MNC";
@@ -103,6 +105,9 @@ public class CarrierKeyDownloadManager { 
    private static final String JSON_TYPE_VALUE_WLAN = "WLAN";

    private static final String JSON_TYPE_VALUE_WLAN = "WLAN";

    private static final String JSON_TYPE_VALUE_EPDG = "EPDG";

    private static final String JSON_TYPE_VALUE_EPDG = "EPDG";





    private static final int EVENT_ALARM_OR_CONFIG_CHANGE = 0;

    private static final int EVENT_DOWNLOAD_COMPLETE = 1;







    private static final int[] CARRIER_KEY_TYPES = {TelephonyManager.KEY_TYPE_EPDG,

    private static final int[] CARRIER_KEY_TYPES = {TelephonyManager.KEY_TYPE_EPDG,

            TelephonyManager.KEY_TYPE_WLAN};

            TelephonyManager.KEY_TYPE_WLAN};
@@ -133,31 +138,43 @@ public class CarrierKeyDownloadManager { 
            int slotId = mPhone.getPhoneId();

            int slotId = mPhone.getPhoneId();

            if (action.equals(INTENT_KEY_RENEWAL_ALARM_PREFIX + slotId)) {

            if (action.equals(INTENT_KEY_RENEWAL_ALARM_PREFIX + slotId)) {

                Log.d(LOG_TAG, "Handling key renewal alarm: " + action);

                Log.d(LOG_TAG, "Handling key renewal alarm: " + action);

                handleAlarmOrConfigChange();

                sendEmptyMessage(EVENT_ALARM_OR_CONFIG_CHANGE);

            } else if (action.equals(TelephonyIntents.ACTION_CARRIER_CERTIFICATE_DOWNLOAD)) {

            } else if (action.equals(TelephonyIntents.ACTION_CARRIER_CERTIFICATE_DOWNLOAD)) {

                if (slotId == intent.getIntExtra(PhoneConstants.PHONE_KEY,

                if (slotId == intent.getIntExtra(PhoneConstants.PHONE_KEY,

                        SubscriptionManager.INVALID_SIM_SLOT_INDEX)) {

                        SubscriptionManager.INVALID_SIM_SLOT_INDEX)) {

                    Log.d(LOG_TAG, "Handling reset intent: " + action);

                    Log.d(LOG_TAG, "Handling reset intent: " + action);

                    handleAlarmOrConfigChange();

                    sendEmptyMessage(EVENT_ALARM_OR_CONFIG_CHANGE);

                }

                }

            } else if (action.equals(CarrierConfigManager.ACTION_CARRIER_CONFIG_CHANGED)) {

            } else if (action.equals(CarrierConfigManager.ACTION_CARRIER_CONFIG_CHANGED)) {

                if (slotId == intent.getIntExtra(PhoneConstants.PHONE_KEY,

                if (slotId == intent.getIntExtra(PhoneConstants.PHONE_KEY,

                        SubscriptionManager.INVALID_SIM_SLOT_INDEX)) {

                        SubscriptionManager.INVALID_SIM_SLOT_INDEX)) {

                    Log.d(LOG_TAG, "Carrier Config changed: " + action);

                    Log.d(LOG_TAG, "Carrier Config changed: " + action);

                    handleAlarmOrConfigChange();

                    sendEmptyMessage(EVENT_ALARM_OR_CONFIG_CHANGE);

                }

                }

            } else if (action.equals(DownloadManager.ACTION_DOWNLOAD_COMPLETE)) {

            } else if (action.equals(DownloadManager.ACTION_DOWNLOAD_COMPLETE)) {

                Log.d(LOG_TAG, "Download Complete");

                Log.d(LOG_TAG, "Download Complete");

                long carrierKeyDownloadIdentifier =

                sendMessage(obtainMessage(EVENT_DOWNLOAD_COMPLETE,

                        intent.getLongExtra(DownloadManager.EXTRA_DOWNLOAD_ID, 0);

                        intent.getLongExtra(DownloadManager.EXTRA_DOWNLOAD_ID, 0)));

            }

        }

    };



    @Override

    public void handleMessage (Message msg) {

        switch (msg.what) {

            case EVENT_ALARM_OR_CONFIG_CHANGE:

                handleAlarmOrConfigChange();

                break;

            case EVENT_DOWNLOAD_COMPLETE:

                long carrierKeyDownloadIdentifier = (long) msg.obj;

                String mccMnc = getMccMncSetFromPref();

                String mccMnc = getMccMncSetFromPref();

                if (isValidDownload(mccMnc)) {

                if (isValidDownload(mccMnc)) {

                    onDownloadComplete(carrierKeyDownloadIdentifier, mccMnc);

                    onDownloadComplete(carrierKeyDownloadIdentifier, mccMnc);

                    onPostDownloadProcessing(carrierKeyDownloadIdentifier);

                    onPostDownloadProcessing(carrierKeyDownloadIdentifier);

                }

                }

                break;

        }

        }

    }

    }

    };





    private void onPostDownloadProcessing(long carrierKeyDownloadIdentifier) {

    private void onPostDownloadProcessing(long carrierKeyDownloadIdentifier) {

        resetRenewalAlarm();

        resetRenewalAlarm();

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

+4 −0
Original line number Original line Diff line number Diff line
@@ -46,6 +46,7 @@ import java.util.GregorianCalendar; 




import static android.preference.PreferenceManager.getDefaultSharedPreferences;

import static android.preference.PreferenceManager.getDefaultSharedPreferences;





import static com.android.internal.telephony.TelephonyTestUtils.waitForMs;

import static org.junit.Assert.assertFalse;

import static org.junit.Assert.assertFalse;

import static org.junit.Assert.assertTrue;

import static org.junit.Assert.assertTrue;

import static org.junit.Assert.fail;

import static org.junit.Assert.fail;
@@ -274,6 +275,7 @@ public class CarrierKeyDownloadMgrTest extends TelephonyTest { 
        when(mTelephonyManager.getSimOperator(anyInt())).thenReturn("310260");

        when(mTelephonyManager.getSimOperator(anyInt())).thenReturn("310260");

        Intent mIntent = new Intent(DownloadManager.ACTION_DOWNLOAD_COMPLETE);

        Intent mIntent = new Intent(DownloadManager.ACTION_DOWNLOAD_COMPLETE);

        mContext.sendBroadcast(mIntent);

        mContext.sendBroadcast(mIntent);

        waitForMs(200);

        Date expirationDate = new Date(mCarrierKeyDM.getExpirationDate());

        Date expirationDate = new Date(mCarrierKeyDM.getExpirationDate());

        assertTrue(dt.format(expirationDate).equals(dateExpected));

        assertTrue(dt.format(expirationDate).equals(dateExpected));

    }

    }
@@ -296,6 +298,7 @@ public class CarrierKeyDownloadMgrTest extends TelephonyTest { 
        Intent mIntent = new Intent(CarrierConfigManager.ACTION_CARRIER_CONFIG_CHANGED);

        Intent mIntent = new Intent(CarrierConfigManager.ACTION_CARRIER_CONFIG_CHANGED);

        mIntent.putExtra(PhoneConstants.PHONE_KEY, 0);

        mIntent.putExtra(PhoneConstants.PHONE_KEY, 0);

        mContext.sendBroadcast(mIntent);

        mContext.sendBroadcast(mIntent);

        waitForMs(200);

        SharedPreferences preferences = getDefaultSharedPreferences(mContext);

        SharedPreferences preferences = getDefaultSharedPreferences(mContext);

        String mccMnc = preferences.getString("CARRIER_KEY_DM_MCC_MNC" + slotId, null);

        String mccMnc = preferences.getString("CARRIER_KEY_DM_MCC_MNC" + slotId, null);

        assertTrue(mccMnc.equals("310:260"));

        assertTrue(mccMnc.equals("310:260"));
@@ -319,6 +322,7 @@ public class CarrierKeyDownloadMgrTest extends TelephonyTest { 
        Intent mIntent = new Intent("com.android.internal.telephony.carrier_key_download_alarm"

        Intent mIntent = new Intent("com.android.internal.telephony.carrier_key_download_alarm"

                + slotId);

                + slotId);

        mContext.sendBroadcast(mIntent);

        mContext.sendBroadcast(mIntent);

        waitForMs(200);

        SharedPreferences preferences = getDefaultSharedPreferences(mContext);

        SharedPreferences preferences = getDefaultSharedPreferences(mContext);

        String mccMnc = preferences.getString("CARRIER_KEY_DM_MCC_MNC" + slotId, null);

        String mccMnc = preferences.getString("CARRIER_KEY_DM_MCC_MNC" + slotId, null);

        assertTrue(mccMnc.equals("310:260"));

        assertTrue(mccMnc.equals("310:260"));