Merge "Fix loading SIM phonebook redundantly" am: b7d3665554 (0ee0391c) · Commits · e / os / android_frameworks_opt_telephony

src/java/com/android/internal/telephony/uicc/AdnCapacity.java

+12 −0

Original line number	Diff line number	Diff line
		@@ -54,6 +54,10 @@ public class AdnCapacity implements Parcelable {
		mMaxAnrLength = maxAnrLength;
		}

		public AdnCapacity() {
		this(0, 0, 0, 0, 0, 0, 0, 0, 0, 0);
		}

		public int getMaxAdnCount() {
		return mMaxAdnCount;
		}
		@@ -98,6 +102,14 @@ public class AdnCapacity implements Parcelable {
		return mMaxAdnCount == mUsedAdnCount;
		}

		public boolean isSimEmpty() {
		return mUsedAdnCount == 0;
		}

		public boolean isSimValid() {
		return mMaxAdnCount > 0;
		}

		public static final Parcelable.Creator<AdnCapacity> CREATOR
		= new Parcelable.Creator<AdnCapacity>() {
		@Override

src/java/com/android/internal/telephony/uicc/SimPhonebookRecordCache.java

+22 −6

Original line number	Diff line number	Diff line
		@@ -54,7 +54,7 @@ import java.util.stream.Collectors;
		value = "TelephonyManager.CAPABILITY_SIM_PHONEBOOK_IN_MODEM")
		public class SimPhonebookRecordCache extends Handler {
		// Instance Variables
		private static final String LOG_TAG = "SimPhonebookRecordCache";
		private String LOG_TAG = "SimPhonebookRecordCache";
		private static final boolean DBG = true;
		// Event Constants
		private static final int EVENT_PHONEBOOK_CHANGED = 1;
		@@ -100,6 +100,7 @@ public class SimPhonebookRecordCache extends Handler {
		mCi = ci;
		mPhoneId = phoneId;
		mContext = context;
		LOG_TAG += "[" + phoneId + "]";
		mCi.registerForSimPhonebookChanged(this, EVENT_PHONEBOOK_CHANGED, null);
		mCi.registerForIccRefresh(this, EVENT_SIM_REFRESH, null);
		mCi.registerForSimPhonebookRecordsReceived(this, EVENT_PHONEBOOK_RECORDS_RECEIVED, null);
		@@ -205,7 +206,7 @@ public class SimPhonebookRecordCache extends Handler {
		synchronized (mReadLock) {
		mAdnLoadingWaiters.add(response);
		final int pendingSize = mAdnLoadingWaiters.size();
		boolean isCapacityInvalid = getAdnCapacity() == null;
		final boolean isCapacityInvalid = isAdnCapacityInvalid();
		if (isCapacityInvalid) {
		getSimPhonebookCapacity();
		}
		@@ -227,6 +228,10 @@ public class SimPhonebookRecordCache extends Handler {
		fillCache();
		}

		private boolean isAdnCapacityInvalid() {
		return getAdnCapacity() == null \|\| !getAdnCapacity().isSimValid();
		}

		@VisibleForTesting
		public boolean isLoading() {
		return mIsRecordLoading.get();
		@@ -312,7 +317,7 @@ public class SimPhonebookRecordCache extends Handler {
		.build();
		UpdateRequest updateRequest = new UpdateRequest(index, newAdn, updateAdn, response);
		mUpdateRequests.add(updateRequest);
		boolean isCapacityInvalid = getAdnCapacity() == null;
		final boolean isCapacityInvalid = isAdnCapacityInvalid();
		if (isCapacityInvalid) {
		getSimPhonebookCapacity();
		}
		@@ -358,6 +363,11 @@ public class SimPhonebookRecordCache extends Handler {
		if (ar != null && ar.exception == null) {
		AdnCapacity capacity = (AdnCapacity)ar.result;
		handlePhonebookCapacityChanged(capacity);
		} else {
		if (!isAdnCapacityInvalid()) {
		mAdnCapacity.set(new AdnCapacity());
		}
		invalidateSimPbCache();
		}
		break;
		case EVENT_PHONEBOOK_RECORDS_RECEIVED:
		@@ -425,10 +435,13 @@ public class SimPhonebookRecordCache extends Handler {

		private void handlePhonebookCapacityChanged(AdnCapacity newCapacity) {
		AdnCapacity oldCapacity = mAdnCapacity.get();
		if (newCapacity == null) {
		newCapacity = new AdnCapacity();
		}
		mAdnCapacity.set(newCapacity);
		if (oldCapacity == null && newCapacity != null) {
		if (newCapacity.getMaxAdnCount() > 0){
		mSimPbRecords.clear();
		if (!newCapacity.isSimEmpty()){
		invalidateSimPbCache();
		fillCacheWithoutWaiting();
		} else {
		notifyAdnLoadingWaiters();
		@@ -436,7 +449,9 @@ public class SimPhonebookRecordCache extends Handler {
		mIsInitialized.set(true); // Let's say the whole process is ready
		} else {
		// There is nothing from PB, so notify waiters directly if any
		if (newCapacity != null && newCapacity.getMaxAdnCount() == 0) {
		if (newCapacity.isSimEmpty()
		\|\| !newCapacity.isSimValid()) {
		mIsCacheInvalidated.set(false);
		notifyAdnLoadingWaiters();
		} else if (!mIsUpdateDone) {
		invalidateSimPbCache();
		@@ -456,6 +471,7 @@ public class SimPhonebookRecordCache extends Handler {
		populateAdnRecords(records.getPhonebookRecords());
		mIsRecordLoading.set(false);
		mIsInRetry.set(false);
		mIsCacheInvalidated.set(false);
		notifyAdnLoadingWaiters();
		tryFireUpdatePendingList();
		} else if (records.isRetryNeeded() && !mIsInRetry.get()) {