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

Commit 9d39cc3d authored by Hall Liu's avatar Hall Liu Committed by Android (Google) Code Review
Browse files

Merge "Replace TelephonyManager methods for mobile data"

parents 6265cf8e d67a1c3d
Loading
Loading
Loading
Loading
+5 −2
Original line number Diff line number Diff line
@@ -90,12 +90,15 @@ public class DataDuringCallsPreferenceController extends TelephonyTogglePreferen

    @Override
    public boolean isChecked() {
        return mManager.isDataAllowedInVoiceCall();
        return mManager.isMobileDataPolicyEnabled(
                TelephonyManager.MOBILE_DATA_POLICY_DATA_ON_NON_DEFAULT_DURING_VOICE_CALL);
    }

    @Override
    public boolean setChecked(boolean isChecked) {
        mManager.setDataAllowedDuringVoiceCall(isChecked);
        mManager.setMobileDataPolicyEnabledStatus(
                TelephonyManager.MOBILE_DATA_POLICY_DATA_ON_NON_DEFAULT_DURING_VOICE_CALL,
                isChecked);
        return true;
    }

+4 −1
Original line number Diff line number Diff line
@@ -88,7 +88,10 @@ public class MmsMessagePreferenceController extends TelephonyTogglePreferenceCon

    @Override
    public boolean setChecked(boolean isChecked) {
        return mTelephonyManager.setAlwaysAllowMmsData(isChecked);
        mTelephonyManager.setMobileDataPolicyEnabledStatus(
                TelephonyManager.MOBILE_DATA_POLICY_MMS_ALWAYS_ALLOWED, isChecked);
        return isChecked == mTelephonyManager.isMobileDataPolicyEnabled(
                TelephonyManager.MOBILE_DATA_POLICY_MMS_ALWAYS_ALLOWED);
    }

    @Override
+4 −2
Original line number Diff line number Diff line
@@ -115,9 +115,11 @@ public class MmsMessagePreferenceControllerTest {
    @Test
    public void setChecked_setDataIntoSubscriptionManager() {
        mController.setChecked(true);
        verify(mTelephonyManager).setAlwaysAllowMmsData(true);
        verify(mTelephonyManager).setMobileDataPolicyEnabledStatus(
                TelephonyManager.MOBILE_DATA_POLICY_MMS_ALWAYS_ALLOWED, true);

        mController.setChecked(false);
        verify(mTelephonyManager).setAlwaysAllowMmsData(false);
        verify(mTelephonyManager).setMobileDataPolicyEnabledStatus(
                TelephonyManager.MOBILE_DATA_POLICY_MMS_ALWAYS_ALLOWED, false);
    }
}
+4 −2
Original line number Diff line number Diff line
@@ -111,9 +111,11 @@ public class MmsMessagePreferenceControllerTest {
    @Test
    public void setChecked_setDataIntoSubscriptionManager() {
        mController.setChecked(true);
        verify(mTelephonyManager).setAlwaysAllowMmsData(true);
        verify(mTelephonyManager).setMobileDataPolicyEnabledStatus(
                TelephonyManager.MOBILE_DATA_POLICY_MMS_ALWAYS_ALLOWED, true);

        mController.setChecked(false);
        verify(mTelephonyManager).setAlwaysAllowMmsData(false);
        verify(mTelephonyManager).setMobileDataPolicyEnabledStatus(
                TelephonyManager.MOBILE_DATA_POLICY_MMS_ALWAYS_ALLOWED, false);
    }
}