[DO NOT MERGE] Best-Effort Load of Potentially-Large SIM Records

Because we have at least one SIM in the field that has a massive
OPLMNw/ACT Record of 700 entries (3.5kB), some messaging
pipelines cannot handle these record sizes and the errors are
preventing the SIM load procedure from being completed.
To work around this, because these records are informational, they
will now load as part of SIM refresh, but that procedure will be
best-effort.

Bug: 34095124
Test: Manually verified with TMO and CT SIMs
Change-Id: I87eb84386a7e8094f3a0493f01ee8dc928864dc2
(cherry picked from commit f15ee16d1f9c325346a002a44b793a2a914ead9e)

1 files changed, 0 insertions, 8 deletions

diff --git a/src/java/com/android/internal/telephony/uicc/SIMRecords.java b/src/java/com/android/internal/telephony/uicc/SIMRecords.java
index 96311b382..d5985c808 100644
--- a/src/java/com/android/internal/telephony/uicc/SIMRecords.java
+++ b/src/java/com/android/internal/telephony/uicc/SIMRecords.java
@@ -1259,7 +1259,6 @@ public class SIMRecords extends IccRecords {
                 break;
 
             case EVENT_GET_PLMN_W_ACT_DONE:
-                isRecordLoadResponse = true;
                 ar = (AsyncResult) msg.obj;
                 data = (byte[]) ar.result;
 
@@ -1274,7 +1273,6 @@ public class SIMRecords extends IccRecords {
                 break;
 
             case EVENT_GET_OPLMN_W_ACT_DONE:
-                isRecordLoadResponse = true;
                 ar = (AsyncResult) msg.obj;
                 data = (byte[]) ar.result;
 
@@ -1290,7 +1288,6 @@ public class SIMRecords extends IccRecords {
                 break;
 
             case EVENT_GET_HPLMN_W_ACT_DONE:
-                isRecordLoadResponse = true;
                 ar = (AsyncResult) msg.obj;
                 data = (byte[]) ar.result;
 
@@ -1305,7 +1302,6 @@ public class SIMRecords extends IccRecords {
                 break;
 
             case EVENT_GET_EHPLMN_DONE:
-                isRecordLoadResponse = true;
                 ar = (AsyncResult) msg.obj;
                 data = (byte[]) ar.result;
                 if (ar.exception != null || data == null) {
@@ -1723,16 +1719,12 @@ public class SIMRecords extends IccRecords {
         mRecordsToLoad++;
 
         mFh.loadEFTransparent(EF_PLMN_W_ACT, obtainMessage(EVENT_GET_PLMN_W_ACT_DONE));
-        mRecordsToLoad++;
 
         mFh.loadEFTransparent(EF_OPLMN_W_ACT, obtainMessage(EVENT_GET_OPLMN_W_ACT_DONE));
-        mRecordsToLoad++;
 
         mFh.loadEFTransparent(EF_HPLMN_W_ACT, obtainMessage(EVENT_GET_HPLMN_W_ACT_DONE));
-        mRecordsToLoad++;
 
         mFh.loadEFTransparent(EF_EHPLMN, obtainMessage(EVENT_GET_EHPLMN_DONE));
-        mRecordsToLoad++;
 
         mFh.loadEFTransparent(EF_FPLMN, obtainMessage(EVENT_GET_FPLMN_DONE));
         mRecordsToLoad++;