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

Commit 520b14de authored by Rambo Wang's avatar Rambo Wang Committed by Automerger Merge Worker
Browse files

Merge "ServiceStateTracker: remove synchronization from mSignalRequestRecords"... 

Merge "ServiceStateTracker: remove synchronization from mSignalRequestRecords" am: a44f53d6 am: 838f8e6c am: 2aa2c23e am: 650f36db

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

MUST ONLY BE SUBMITTED BY AUTOMERGER

Change-Id: Ie2ca45a97c18efc5e6f6c5e75cccd5fd96d8932e
parents 63d25d20 650f36db

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

+20 −28
Original line number Diff line number Diff line
@@ -654,7 +654,6 @@ public class ServiceStateTracker extends Handler { 
    private final Object mLteRsrpBoostLock = new Object();

    private static final int INVALID_LTE_EARFCN = -1;



    // @GuardedBy("mSignalRequestRecords")

    private final List<SignalRequestRecord> mSignalRequestRecords = new ArrayList<>();



    public ServiceStateTracker(GsmCdmaPhone phone, CommandsInterface ci) {
@@ -1795,10 +1794,7 @@ public class ServiceStateTracker extends Handler { 
                    break;

                }



                synchronized (mSignalRequestRecords) {

                mSignalRequestRecords.add(record);

                }



                updateAlwaysReportSignalStrength();

                updateReportingCriteria(getCarrierConfig());
@@ -1812,7 +1808,6 @@ public class ServiceStateTracker extends Handler { 
                SignalRequestRecord record = pair.first;

                Message onCompleted = pair.second;



                synchronized (mSignalRequestRecords) {

                // for loop with removal may cause ConcurrentModificationException

                Iterator<SignalRequestRecord> it = mSignalRequestRecords.iterator();

                while (it.hasNext()) {
@@ -1821,7 +1816,6 @@ public class ServiceStateTracker extends Handler { 
                        it.remove();

                    }

                }

                }



                updateAlwaysReportSignalStrength();

                updateReportingCriteria(getCarrierConfig());
@@ -6136,7 +6130,6 @@ public class ServiceStateTracker extends Handler { 


        final boolean isDeviceIdle = mPhone.isDeviceIdle();

        final int curSubId = mPhone.getSubId();

        synchronized (mSignalRequestRecords) {

        // The total number of record is small (10~15 tops). With each request has at most 5

        // SignalThresholdInfo which has at most 8 thresholds arrays. So the nested loop should

        // not be a concern here.
@@ -6154,7 +6147,6 @@ public class ServiceStateTracker extends Handler { 
                }

            }

        }

        }



        int[] targetArray = new int[target.size()];

        int i = 0;