From d270f29cc893551ab99dc082db45f03f413613c1 Mon Sep 17 00:00:00 2001 From: Jack Yu Date: Thu, 4 Aug 2022 09:52:30 +0000 Subject: [PATCH] Show reboot dialog when users disable developer options When NFC detailed logging mechanisms are enabled and users try to disable develoepr options, show reboot request dialog to make sure NFC logging mechanisms are also disabled. Bug: 204397062 Test: manual Change-Id: I531c85fd77934ad7821c480df07465a8cd8f24f9 --- .../DevelopmentSettingsDashboardFragment.java | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/src/com/android/settings/development/DevelopmentSettingsDashboardFragment.java b/src/com/android/settings/development/DevelopmentSettingsDashboardFragment.java index 856bee37e26..210d0115334 100644 --- a/src/com/android/settings/development/DevelopmentSettingsDashboardFragment.java +++ b/src/com/android/settings/development/DevelopmentSettingsDashboardFragment.java @@ -300,10 +300,19 @@ public class DevelopmentSettingsDashboardFragment extends RestrictedDashboardFra final BluetoothLeAudioHwOffloadPreferenceController leAudioController = getDevelopmentOptionsController( BluetoothLeAudioHwOffloadPreferenceController.class); + final NfcSnoopLogPreferenceController nfcSnoopLogController = + getDevelopmentOptionsController( + NfcSnoopLogPreferenceController.class); + final NfcVerboseVendorLogPreferenceController nfcVerboseLogController = + getDevelopmentOptionsController( + NfcVerboseVendorLogPreferenceController.class); // If hardware offload isn't default value, we must reboot after disable // developer options. Show a dialog for the user to confirm. if ((a2dpController == null || a2dpController.isDefaultValue()) - && (leAudioController == null || leAudioController.isDefaultValue())) { + && (leAudioController == null || leAudioController.isDefaultValue()) + && (nfcSnoopLogController == null || nfcSnoopLogController.isDefaultValue()) + && (nfcVerboseLogController == null + || nfcVerboseLogController.isDefaultValue())) { disableDeveloperOptions(); } else { DisableDevSettingsDialogFragment.show(this /* host */);