Merge "Do isAvailable() checks when accessing AirplaneEnabler." into udc-dev am: b232ba29e6
Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/apps/Settings/+/21725541 Change-Id: Ic9f27eb0e6eba63b719edcb31a0cc54909d4fd8b Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
@@ -79,7 +79,7 @@ public class AirplaneModePreferenceController extends TogglePreferenceController
|
||||
|
||||
@Override
|
||||
public boolean handlePreferenceTreeClick(Preference preference) {
|
||||
if (KEY_AIRPLANE_MODE.equals(preference.getKey())
|
||||
if (KEY_AIRPLANE_MODE.equals(preference.getKey()) && isAvailable()
|
||||
&& mAirplaneModeEnabler.isInEcmMode()) {
|
||||
// In ECM mode launch ECM app dialog
|
||||
if (mFragment != null) {
|
||||
@@ -141,12 +141,14 @@ public class AirplaneModePreferenceController extends TogglePreferenceController
|
||||
|
||||
@Override
|
||||
public void onDestroy() {
|
||||
mAirplaneModeEnabler.close();
|
||||
if (isAvailable()) {
|
||||
mAirplaneModeEnabler.close();
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
public void onActivityResult(int requestCode, int resultCode, Intent data) {
|
||||
if (requestCode == REQUEST_CODE_EXIT_ECM) {
|
||||
if (requestCode == REQUEST_CODE_EXIT_ECM && isAvailable()) {
|
||||
final boolean isChoiceYes = (resultCode == Activity.RESULT_OK);
|
||||
// Set Airplane mode based on the return value and checkbox state
|
||||
mAirplaneModeEnabler.setAirplaneModeInECM(isChoiceYes,
|
||||
@@ -156,7 +158,7 @@ public class AirplaneModePreferenceController extends TogglePreferenceController
|
||||
|
||||
@Override
|
||||
public boolean isChecked() {
|
||||
return mAirplaneModeEnabler.isAirplaneModeOn();
|
||||
return isAvailable() && mAirplaneModeEnabler.isAirplaneModeOn();
|
||||
}
|
||||
|
||||
@Override
|
||||
@@ -164,7 +166,9 @@ public class AirplaneModePreferenceController extends TogglePreferenceController
|
||||
if (isChecked() == isChecked) {
|
||||
return false;
|
||||
}
|
||||
mAirplaneModeEnabler.setAirplaneMode(isChecked);
|
||||
if (isAvailable()) {
|
||||
mAirplaneModeEnabler.setAirplaneMode(isChecked);
|
||||
}
|
||||
return true;
|
||||
}
|
||||
|
||||
|
Reference in New Issue
Block a user