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

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

Merge "Clear up the obsoleted cascading signal strength polling logic" am:... 

Merge "Clear up the obsoleted cascading signal strength polling logic" am: eddd69f2 am: 7ab8138a am: 33466fa6

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

Change-Id: I3ee0efc1c9e15e3e4c234ec7c1bf302b7e5cda97
parents 1a5f6147 33466fa6

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

+0 −9
Original line number Diff line number Diff line
@@ -1284,8 +1284,6 @@ public class ServiceStateTracker extends Handler { 
                mPrevSubId = SubscriptionManager.INVALID_SUBSCRIPTION_ID;

                mIsSimReady = true;

                pollStateInternal(false);

                // Signal strength polling stops when radio is off

                queueNextSignalStrengthPoll();

                break;



            case EVENT_RADIO_STATE_CHANGED:
@@ -1293,9 +1291,6 @@ public class ServiceStateTracker extends Handler { 
                if(!mPhone.isPhoneTypeGsm() &&

                        mCi.getRadioState() == TelephonyManager.RADIO_POWER_ON) {

                    handleCdmaSubscriptionSource(mCdmaSSM.getCdmaSubscriptionSource());



                    // Signal strength polling stops when radio is off.

                    queueNextSignalStrengthPoll();

                }

                // This will do nothing in the 'radio not available' case

                setPowerStateToDesired();
@@ -4607,10 +4602,6 @@ public class ServiceStateTracker extends Handler { 
        }

    }



    private void queueNextSignalStrengthPoll() {

        mPhone.getSignalStrengthController().queueNextSignalStrengthPoll();

    }



    private void notifyCdmaSubscriptionInfoReady() {

        if (mCdmaForSubscriptionInfoReadyRegistrants != null) {

            if (DBG) log("CDMA_SUBSCRIPTION: call notifyRegistrants()");

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

+0 −38
Original line number Diff line number Diff line
@@ -38,14 +38,10 @@ import android.telephony.SignalStrength; 
import android.telephony.SignalStrengthUpdateRequest;

import android.telephony.SignalThresholdInfo;

import android.telephony.SubscriptionInfo;

import android.telephony.SubscriptionManager;

import android.telephony.TelephonyManager;

import android.util.Pair;



import com.android.internal.annotations.VisibleForTesting;

import com.android.internal.telephony.uicc.IccCardStatus;

import com.android.internal.telephony.uicc.UiccCard;

import com.android.internal.telephony.uicc.UiccController;

import com.android.internal.telephony.util.ArrayUtils;

import com.android.internal.util.IndentingPrintWriter;

import com.android.telephony.Rlog;
@@ -86,12 +82,6 @@ public class SignalStrengthController extends Handler { 
    private final Phone mPhone;

    private final CommandsInterface mCi;



    /**

     * By default, strength polling is enabled.  However, if we're

     * getting unsolicited signal strength updates from the radio, set

     * value to true and don't bother polling any more.

     */

    private boolean mDontPollSignalStrength = false;

    @NonNull

    private SignalStrength mSignalStrength;

    private long mSignalStrengthUpdatedTime;
@@ -215,8 +205,6 @@ public class SignalStrengthController extends Handler { 
                }

                ar = (AsyncResult) msg.obj;

                onSignalStrengthResult(ar);

                queueNextSignalStrengthPoll();



                break;

            }
@@ -231,11 +219,6 @@ public class SignalStrengthController extends Handler { 
                // This is a notification from CommandsInterface.setOnSignalStrengthUpdate



                ar = (AsyncResult) msg.obj;



                // The radio is telling us about signal strength changes

                // we don't have to ask it

                mDontPollSignalStrength = true;



                onSignalStrengthResult(ar);

                break;

            }
@@ -335,26 +318,6 @@ public class SignalStrengthController extends Handler { 
        return false;

    }



    void queueNextSignalStrengthPoll() {

        if (mDontPollSignalStrength) {

            // The radio is telling us about signal strength changes

            // we don't have to ask it

            return;

        }



        // if there is no SIM present, do not poll signal strength

        UiccCard uiccCard = UiccController.getInstance().getUiccCard(

                mPhone != null ? mPhone.getPhoneId() : SubscriptionManager.DEFAULT_PHONE_INDEX);

        if (uiccCard == null

                || uiccCard.getCardState() == IccCardStatus.CardState.CARDSTATE_ABSENT) {

            log("Not polling signal strength due to absence of SIM");

            return;

        }



        // TODO Don't poll signal strength if screen is off

        sendMessageDelayed(obtainMessage(EVENT_POLL_SIGNAL_STRENGTH), POLL_PERIOD_MILLIS);

    }



    /**

     * Update signal strength reporting criteria from the carrier config

     */
@@ -477,7 +440,6 @@ public class SignalStrengthController extends Handler { 
        pw.println("mSignalRequestRecords=" + mSignalRequestRecords);

        pw.println(" mLastSignalStrength=" + mLastSignalStrength);

        pw.println(" mSignalStrength=" + mSignalStrength);

        pw.println(" mDontPollSignalStrength=" + mDontPollSignalStrength);

        pw.println(" mLteRsrpBoost=" + mLteRsrpBoost);

        pw.println(" mNrRsrpBoost=" + Arrays.toString(mNrRsrpBoost));

        dumpEarfcnPairList(pw, mEarfcnPairListForRsrpBoost, "mEarfcnPairListForRsrpBoost");