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

Commit 3b9e4e1f authored by Nazanin Bakhshi's avatar Nazanin Bakhshi
Browse files

Send an async request to modem each time getModemStatus is called to eventually 

update the modemStatus, and not rely on stale cached value

Bug: 135067156
Test: manual
Change-Id: Iaf9b47c7c649ad031389278e2be8b3bc65f2ad58
Merged-In: Iaf9b47c7c649ad031389278e2be8b3bc65f2ad58
(cherry picked from commit a79b4c9c)
parent 773a14b1

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

+3 −1
Original line number Diff line number Diff line
@@ -208,12 +208,14 @@ public class PhoneConfigurationManager { 
        try {

            return getPhoneStatusFromCache(phoneId);

        } catch (NoSuchElementException ex) {

            updatePhoneStatus(phone);

            // Return true if modem status cannot be retrieved. For most cases, modem status

            // is on. And for older version modems, GET_MODEM_STATUS and disable modem are not

            // supported. Modem is always on.

            //TODO: this should be fixed in R to support a third status UNKNOWN b/131631629

            return true;

        } finally {

            //in either case send an asynchronous request to retrieve the phone status

            updatePhoneStatus(phone);

        }

    }