diff --git a/src/com/android/settings/accessibility/CaptioningTogglePreferenceController.java b/src/com/android/settings/accessibility/CaptioningTogglePreferenceController.java index a9d2cdfafb9..10e1fcae330 100644 --- a/src/com/android/settings/accessibility/CaptioningTogglePreferenceController.java +++ b/src/com/android/settings/accessibility/CaptioningTogglePreferenceController.java @@ -17,18 +17,18 @@ package com.android.settings.accessibility; import android.content.Context; -import android.widget.Switch; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import androidx.preference.PreferenceScreen; import com.android.settings.R; import com.android.settings.core.TogglePreferenceController; import com.android.settings.widget.SettingsMainSwitchPreference; -import com.android.settingslib.widget.OnMainSwitchChangeListener; /** Preference controller for captioning more options. */ public class CaptioningTogglePreferenceController extends TogglePreferenceController - implements OnMainSwitchChangeListener { + implements OnCheckedChangeListener { private final CaptionHelper mCaptionHelper; @@ -63,7 +63,7 @@ public class CaptioningTogglePreferenceController extends TogglePreferenceContro } @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { if (isChecked != isChecked()) { setChecked(isChecked); } diff --git a/src/com/android/settings/accessibility/ToggleAccessibilityServicePreferenceFragment.java b/src/com/android/settings/accessibility/ToggleAccessibilityServicePreferenceFragment.java index 28b80baa32e..6847a6dc3c1 100644 --- a/src/com/android/settings/accessibility/ToggleAccessibilityServicePreferenceFragment.java +++ b/src/com/android/settings/accessibility/ToggleAccessibilityServicePreferenceFragment.java @@ -41,7 +41,7 @@ import android.view.Menu; import android.view.MenuInflater; import android.view.View; import android.view.accessibility.AccessibilityManager; -import android.widget.Switch; +import android.widget.CompoundButton; import androidx.annotation.Nullable; @@ -294,7 +294,7 @@ public class ToggleAccessibilityServicePreferenceFragment extends } @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { if (isChecked != isAccessibilityServiceEnabled()) { onPreferenceClick(isChecked); } diff --git a/src/com/android/settings/accessibility/ToggleFeaturePreferenceFragment.java b/src/com/android/settings/accessibility/ToggleFeaturePreferenceFragment.java index 3dc0b477e77..427cad9df8d 100644 --- a/src/com/android/settings/accessibility/ToggleFeaturePreferenceFragment.java +++ b/src/com/android/settings/accessibility/ToggleFeaturePreferenceFragment.java @@ -44,8 +44,9 @@ import android.view.ViewGroup; import android.view.accessibility.AccessibilityManager; import android.view.accessibility.AccessibilityManager.TouchExplorationStateChangeListener; import android.widget.CheckBox; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import android.widget.ImageView; -import android.widget.Switch; import androidx.annotation.Nullable; import androidx.annotation.VisibleForTesting; @@ -64,7 +65,6 @@ import com.android.settings.utils.LocaleUtils; import com.android.settings.widget.SettingsMainSwitchBar; import com.android.settings.widget.SettingsMainSwitchPreference; import com.android.settingslib.widget.IllustrationPreference; -import com.android.settingslib.widget.OnMainSwitchChangeListener; import com.android.settingslib.widget.TopIntroPreference; import com.google.android.setupcompat.util.WizardManagerHelper; @@ -78,7 +78,7 @@ import java.util.Locale; * and dialog management. */ public abstract class ToggleFeaturePreferenceFragment extends DashboardFragment - implements ShortcutPreference.OnClickCallback, OnMainSwitchChangeListener { + implements ShortcutPreference.OnClickCallback, OnCheckedChangeListener { public static final String KEY_GENERAL_CATEGORY = "general_categories"; public static final String KEY_SHORTCUT_PREFERENCE = "shortcut_preference"; @@ -328,7 +328,7 @@ public abstract class ToggleFeaturePreferenceFragment extends DashboardFragment } @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { onPreferenceToggled(mPreferenceKey, isChecked); } diff --git a/src/com/android/settings/accounts/WorkModePreferenceController.java b/src/com/android/settings/accounts/WorkModePreferenceController.java index 46440f6690f..ae910f7818d 100644 --- a/src/com/android/settings/accounts/WorkModePreferenceController.java +++ b/src/com/android/settings/accounts/WorkModePreferenceController.java @@ -16,7 +16,8 @@ package com.android.settings.accounts; import android.content.Context; -import android.widget.Switch; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import androidx.lifecycle.DefaultLifecycleObserver; import androidx.lifecycle.LifecycleOwner; @@ -27,14 +28,13 @@ import com.android.settings.R; import com.android.settings.core.BasePreferenceController; import com.android.settings.slices.SliceData; import com.android.settingslib.widget.MainSwitchPreference; -import com.android.settingslib.widget.OnMainSwitchChangeListener; import org.jetbrains.annotations.NotNull; /** Controller for "Work apps" toggle that allows the user to enable/disable quiet mode. */ public class WorkModePreferenceController extends BasePreferenceController - implements OnMainSwitchChangeListener, DefaultLifecycleObserver, + implements OnCheckedChangeListener, DefaultLifecycleObserver, ManagedProfileQuietModeEnabler.QuietModeChangeListener { private final ManagedProfileQuietModeEnabler mQuietModeEnabler; @@ -68,7 +68,7 @@ public class WorkModePreferenceController extends BasePreferenceController } @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { mQuietModeEnabler.setQuietModeEnabled(!isChecked); } diff --git a/src/com/android/settings/applications/intentpicker/AppLaunchSettings.java b/src/com/android/settings/applications/intentpicker/AppLaunchSettings.java index cebb4f78689..72f754379aa 100644 --- a/src/com/android/settings/applications/intentpicker/AppLaunchSettings.java +++ b/src/com/android/settings/applications/intentpicker/AppLaunchSettings.java @@ -35,7 +35,8 @@ import android.util.ArraySet; import android.util.Log; import android.view.LayoutInflater; import android.view.View; -import android.widget.Switch; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import android.widget.TextView; import androidx.annotation.VisibleForTesting; @@ -51,7 +52,6 @@ import com.android.settings.widget.EntityHeaderController; import com.android.settingslib.applications.AppUtils; import com.android.settingslib.widget.FooterPreference; import com.android.settingslib.widget.MainSwitchPreference; -import com.android.settingslib.widget.OnMainSwitchChangeListener; import java.util.HashMap; import java.util.List; @@ -62,7 +62,7 @@ import java.util.UUID; /** The page of the Open by default */ public class AppLaunchSettings extends AppInfoBase implements - Preference.OnPreferenceChangeListener, OnMainSwitchChangeListener { + Preference.OnPreferenceChangeListener, OnCheckedChangeListener { private static final String TAG = "AppLaunchSettings"; // Preference keys private static final String MAIN_SWITCH_PREF_KEY = "open_by_default_supported_links"; @@ -168,7 +168,7 @@ public class AppLaunchSettings extends AppInfoBase implements } @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { IntentPickerUtils.logd("onSwitchChanged: isChecked=" + isChecked); if (mMainSwitchPreference != null) { //mMainSwitchPreference synced with Switch mMainSwitchPreference.setChecked(isChecked); diff --git a/src/com/android/settings/backup/AutoRestorePreferenceController.java b/src/com/android/settings/backup/AutoRestorePreferenceController.java index bf63e25b522..1394ce13619 100644 --- a/src/com/android/settings/backup/AutoRestorePreferenceController.java +++ b/src/com/android/settings/backup/AutoRestorePreferenceController.java @@ -25,7 +25,7 @@ import android.provider.Settings; import android.util.Log; import androidx.preference.Preference; -import androidx.preference.SwitchPreference; +import androidx.preference.TwoStatePreference; import com.android.settings.R; import com.android.settings.core.TogglePreferenceController; @@ -79,7 +79,7 @@ public class AutoRestorePreferenceController extends TogglePreferenceController backupManager.setAutoRestore(nextValue); result = true; } catch (RemoteException e) { - ((SwitchPreference) mPreference).setChecked(!nextValue); + ((TwoStatePreference) mPreference).setChecked(!nextValue); Log.e(TAG, "Error can't set setAutoRestore", e); } diff --git a/src/com/android/settings/backup/ToggleBackupSettingFragment.java b/src/com/android/settings/backup/ToggleBackupSettingFragment.java index 8f60be9a1cf..c3ad92fe23b 100644 --- a/src/com/android/settings/backup/ToggleBackupSettingFragment.java +++ b/src/com/android/settings/backup/ToggleBackupSettingFragment.java @@ -11,7 +11,6 @@ import android.os.ServiceManager; import android.provider.Settings; import android.util.Log; import android.view.View; -import android.widget.Switch; import android.widget.TextView; import androidx.appcompat.app.AlertDialog; @@ -116,8 +115,7 @@ public class ToggleBackupSettingFragment extends SettingsPreferenceFragment mSwitchBar.setOnBeforeCheckedChangeListener( new SettingsMainSwitchBar.OnBeforeCheckedChangeListener() { @Override - public boolean onBeforeCheckedChanged( - Switch toggleSwitch, boolean checked) { + public boolean onBeforeCheckedChanged(boolean checked) { if (!checked) { // Don't change Switch status until user makes choice in dialog // so return true here. diff --git a/src/com/android/settings/connecteddevice/audiosharing/AudioSharingSwitchBarController.java b/src/com/android/settings/connecteddevice/audiosharing/AudioSharingSwitchBarController.java index ff383a7269d..83367ae9c11 100644 --- a/src/com/android/settings/connecteddevice/audiosharing/AudioSharingSwitchBarController.java +++ b/src/com/android/settings/connecteddevice/audiosharing/AudioSharingSwitchBarController.java @@ -25,7 +25,8 @@ import android.bluetooth.BluetoothLeBroadcastReceiveState; import android.bluetooth.BluetoothProfile; import android.content.Context; import android.util.Log; -import android.widget.Switch; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import androidx.annotation.NonNull; import androidx.lifecycle.DefaultLifecycleObserver; @@ -42,7 +43,6 @@ import com.android.settingslib.bluetooth.LocalBluetoothLeBroadcast; import com.android.settingslib.bluetooth.LocalBluetoothLeBroadcastAssistant; import com.android.settingslib.bluetooth.LocalBluetoothManager; import com.android.settingslib.utils.ThreadUtils; -import com.android.settingslib.widget.OnMainSwitchChangeListener; import com.google.common.collect.ImmutableList; @@ -55,7 +55,7 @@ import java.util.concurrent.Executor; import java.util.concurrent.Executors; public class AudioSharingSwitchBarController extends BasePreferenceController - implements DefaultLifecycleObserver, OnMainSwitchChangeListener { + implements DefaultLifecycleObserver, OnCheckedChangeListener { private static final String TAG = "AudioSharingSwitchBarCtl"; private static final String PREF_KEY = "audio_sharing_main_switch"; private final SettingsMainSwitchBar mSwitchBar; @@ -229,9 +229,9 @@ public class AudioSharingSwitchBarController extends BasePreferenceController } @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { // Filter out unnecessary callbacks when switch is disabled. - if (!switchView.isEnabled()) return; + if (!buttonView.isEnabled()) return; if (isChecked) { startAudioSharing(); } else { diff --git a/src/com/android/settings/datausage/DataSaverSummary.kt b/src/com/android/settings/datausage/DataSaverSummary.kt index 0828d362410..cdd46f2fa40 100644 --- a/src/com/android/settings/datausage/DataSaverSummary.kt +++ b/src/com/android/settings/datausage/DataSaverSummary.kt @@ -19,7 +19,6 @@ import android.app.settings.SettingsEnums import android.content.Context import android.os.Bundle import android.telephony.SubscriptionManager -import android.widget.Switch import com.android.settings.R import com.android.settings.SettingsActivity import com.android.settings.dashboard.DashboardFragment @@ -51,9 +50,7 @@ class DataSaverSummary : DashboardFragment() { switchBar = (activity as SettingsActivity).switchBar.apply { setTitle(getString(R.string.data_saver_switch_title)) show() - addOnSwitchChangeListener { _: Switch, isChecked: Boolean -> - onSwitchChanged(isChecked) - } + addOnSwitchChangeListener { _, isChecked -> onSwitchChanged(isChecked) } } } @@ -105,4 +102,4 @@ class DataSaverSummary : DashboardFragment() { SubscriptionManager.INVALID_SUBSCRIPTION_ID) } } -} \ No newline at end of file +} diff --git a/src/com/android/settings/deletionhelper/AutomaticStorageManagerSwitchBarController.java b/src/com/android/settings/deletionhelper/AutomaticStorageManagerSwitchBarController.java index f46bb0d18c9..a8cb9ad00ab 100644 --- a/src/com/android/settings/deletionhelper/AutomaticStorageManagerSwitchBarController.java +++ b/src/com/android/settings/deletionhelper/AutomaticStorageManagerSwitchBarController.java @@ -20,7 +20,8 @@ import android.app.settings.SettingsEnums; import android.content.Context; import android.os.SystemProperties; import android.provider.Settings; -import android.widget.Switch; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import androidx.fragment.app.FragmentManager; import androidx.preference.Preference; @@ -29,11 +30,10 @@ import com.android.internal.util.Preconditions; import com.android.settings.widget.SettingsMainSwitchBar; import com.android.settingslib.Utils; import com.android.settingslib.core.instrumentation.MetricsFeatureProvider; -import com.android.settingslib.widget.OnMainSwitchChangeListener; /** Handles the logic for flipping the storage management toggle on a {@link SwitchBar}. */ public class AutomaticStorageManagerSwitchBarController - implements OnMainSwitchChangeListener { + implements OnCheckedChangeListener { private static final String STORAGE_MANAGER_ENABLED_BY_DEFAULT_PROPERTY = "ro.storage_manager.enabled"; @@ -64,7 +64,7 @@ public class AutomaticStorageManagerSwitchBarController } @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { mMetrics.action(mContext, SettingsEnums.ACTION_TOGGLE_STORAGE_MANAGER, isChecked); mDaysToRetainPreference.setEnabled(isChecked); Settings.Secure.putInt( diff --git a/src/com/android/settings/development/DevelopmentSettingsDashboardFragment.java b/src/com/android/settings/development/DevelopmentSettingsDashboardFragment.java index a995ac3c2fd..9c8c186675b 100644 --- a/src/com/android/settings/development/DevelopmentSettingsDashboardFragment.java +++ b/src/com/android/settings/development/DevelopmentSettingsDashboardFragment.java @@ -45,7 +45,8 @@ import android.util.Log; import android.view.LayoutInflater; import android.view.View; import android.view.ViewGroup; -import android.widget.Switch; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import androidx.annotation.VisibleForTesting; import androidx.localbroadcastmanager.content.LocalBroadcastManager; @@ -80,7 +81,6 @@ import com.android.settingslib.development.DeveloperOptionsPreferenceController; import com.android.settingslib.development.DevelopmentSettingsEnabler; import com.android.settingslib.development.SystemPropPoker; import com.android.settingslib.search.SearchIndexable; -import com.android.settingslib.widget.OnMainSwitchChangeListener; import com.google.android.setupcompat.util.WizardManagerHelper; @@ -89,7 +89,7 @@ import java.util.List; @SearchIndexable(forTarget = SearchIndexable.ALL & ~SearchIndexable.ARC) public class DevelopmentSettingsDashboardFragment extends RestrictedDashboardFragment - implements OnMainSwitchChangeListener, OemUnlockDialogHost, AdbDialogHost, + implements OnCheckedChangeListener, OemUnlockDialogHost, AdbDialogHost, AdbClearKeysDialogHost, LogPersistDialogHost, BluetoothRebootDialog.OnRebootDialogListener, AbstractBluetoothPreferenceController.Callback, @@ -333,7 +333,7 @@ public class DevelopmentSettingsDashboardFragment extends RestrictedDashboardFra } @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { final boolean developmentEnabledState = DevelopmentSettingsEnabler.isDevelopmentSettingsEnabled(getContext()); if (isChecked != developmentEnabledState) { diff --git a/src/com/android/settings/display/AutoBrightnessDetailPreferenceController.java b/src/com/android/settings/display/AutoBrightnessDetailPreferenceController.java index ec311f400f2..9bfb2f4e222 100644 --- a/src/com/android/settings/display/AutoBrightnessDetailPreferenceController.java +++ b/src/com/android/settings/display/AutoBrightnessDetailPreferenceController.java @@ -17,18 +17,18 @@ package com.android.settings.display; import android.content.Context; -import android.widget.Switch; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import androidx.preference.PreferenceScreen; import com.android.settings.widget.SettingsMainSwitchPreference; -import com.android.settingslib.widget.OnMainSwitchChangeListener; /** * Controller that updates the adaptive brightness. */ public class AutoBrightnessDetailPreferenceController extends - AutoBrightnessPreferenceController implements OnMainSwitchChangeListener { + AutoBrightnessPreferenceController implements OnCheckedChangeListener { public AutoBrightnessDetailPreferenceController(Context context, String key) { super(context, key); @@ -58,7 +58,7 @@ public class AutoBrightnessDetailPreferenceController extends } @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { if (isChecked != isChecked()) { setChecked(isChecked); } diff --git a/src/com/android/settings/display/NightDisplayActivationPreferenceController.java b/src/com/android/settings/display/NightDisplayActivationPreferenceController.java index c94f44331dc..fd6d199a29d 100644 --- a/src/com/android/settings/display/NightDisplayActivationPreferenceController.java +++ b/src/com/android/settings/display/NightDisplayActivationPreferenceController.java @@ -19,7 +19,8 @@ package com.android.settings.display; import android.content.Context; import android.hardware.display.ColorDisplayManager; import android.text.TextUtils; -import android.widget.Switch; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import androidx.preference.Preference; import androidx.preference.PreferenceScreen; @@ -29,13 +30,12 @@ import com.android.settings.core.TogglePreferenceController; import com.android.settings.overlay.FeatureFactory; import com.android.settingslib.core.instrumentation.MetricsFeatureProvider; import com.android.settingslib.widget.MainSwitchPreference; -import com.android.settingslib.widget.OnMainSwitchChangeListener; /** * Controller that updates the night display. */ public class NightDisplayActivationPreferenceController extends - TogglePreferenceController implements OnMainSwitchChangeListener { + TogglePreferenceController implements OnCheckedChangeListener { private final MetricsFeatureProvider mMetricsFeatureProvider; private ColorDisplayManager mColorDisplayManager; @@ -81,7 +81,7 @@ public class NightDisplayActivationPreferenceController extends } @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { final boolean activated = mColorDisplayManager.isNightDisplayActivated(); if (isChecked != activated) { // TODO(b/179017365): Create a controller which extends TogglePreferenceController to diff --git a/src/com/android/settings/display/darkmode/DarkModeActivationPreferenceController.java b/src/com/android/settings/display/darkmode/DarkModeActivationPreferenceController.java index eac714bf5c3..5f441be79b8 100644 --- a/src/com/android/settings/display/darkmode/DarkModeActivationPreferenceController.java +++ b/src/com/android/settings/display/darkmode/DarkModeActivationPreferenceController.java @@ -18,7 +18,8 @@ package com.android.settings.display.darkmode; import android.app.UiModeManager; import android.content.Context; import android.content.res.Configuration; -import android.widget.Switch; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import androidx.preference.Preference; import androidx.preference.PreferenceScreen; @@ -28,7 +29,6 @@ import com.android.settings.core.BasePreferenceController; import com.android.settings.overlay.FeatureFactory; import com.android.settingslib.core.instrumentation.MetricsFeatureProvider; import com.android.settingslib.widget.MainSwitchPreference; -import com.android.settingslib.widget.OnMainSwitchChangeListener; import java.time.LocalTime; @@ -36,7 +36,7 @@ import java.time.LocalTime; * Controller for activate/deactivate night mode button */ public class DarkModeActivationPreferenceController extends BasePreferenceController implements - OnMainSwitchChangeListener { + OnCheckedChangeListener { private final UiModeManager mUiModeManager; private final MetricsFeatureProvider mMetricsFeatureProvider; @@ -96,7 +96,7 @@ public class DarkModeActivationPreferenceController extends BasePreferenceContro } @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { mMetricsFeatureProvider.logClickedPreference(mPreference, getMetricsCategory()); final boolean active = (mContext.getResources().getConfiguration().uiMode & Configuration.UI_MODE_NIGHT_YES) != 0; diff --git a/src/com/android/settings/dream/DreamSettings.java b/src/com/android/settings/dream/DreamSettings.java index 25825eac4f7..418bb6418c1 100644 --- a/src/com/android/settings/dream/DreamSettings.java +++ b/src/com/android/settings/dream/DreamSettings.java @@ -29,7 +29,8 @@ import android.view.LayoutInflater; import android.view.View; import android.view.ViewGroup; import android.widget.Button; -import android.widget.Switch; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import androidx.annotation.VisibleForTesting; import androidx.preference.Preference; @@ -44,13 +45,12 @@ import com.android.settingslib.dream.DreamBackend; import com.android.settingslib.dream.DreamBackend.WhenToDream; import com.android.settingslib.search.SearchIndexable; import com.android.settingslib.widget.MainSwitchPreference; -import com.android.settingslib.widget.OnMainSwitchChangeListener; import java.util.ArrayList; import java.util.List; @SearchIndexable -public class DreamSettings extends DashboardFragment implements OnMainSwitchChangeListener { +public class DreamSettings extends DashboardFragment implements OnCheckedChangeListener { private static final String TAG = "DreamSettings"; static final String WHILE_CHARGING_ONLY = "while_charging_only"; @@ -246,7 +246,7 @@ public class DreamSettings extends DashboardFragment implements OnMainSwitchChan } @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { setAllPreferencesEnabled(isChecked); mPreviewButton.setVisibility(isChecked ? View.VISIBLE : View.GONE); updatePaddingForPreviewButton(); diff --git a/src/com/android/settings/emergency/EmergencyGesturePreferenceController.java b/src/com/android/settings/emergency/EmergencyGesturePreferenceController.java index b6135ab2740..b222cc170c7 100644 --- a/src/com/android/settings/emergency/EmergencyGesturePreferenceController.java +++ b/src/com/android/settings/emergency/EmergencyGesturePreferenceController.java @@ -17,7 +17,8 @@ package com.android.settings.emergency; import android.content.Context; -import android.widget.Switch; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import androidx.annotation.VisibleForTesting; import androidx.preference.Preference; @@ -27,13 +28,12 @@ import com.android.settings.R; import com.android.settings.core.BasePreferenceController; import com.android.settingslib.emergencynumber.EmergencyNumberUtils; import com.android.settingslib.widget.MainSwitchPreference; -import com.android.settingslib.widget.OnMainSwitchChangeListener; /** * Preference controller for emergency gesture setting */ public class EmergencyGesturePreferenceController extends BasePreferenceController implements - OnMainSwitchChangeListener { + OnCheckedChangeListener { @VisibleForTesting EmergencyNumberUtils mEmergencyNumberUtils; @@ -72,7 +72,7 @@ public class EmergencyGesturePreferenceController extends BasePreferenceControll } @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { mEmergencyNumberUtils.setEmergencyGestureEnabled(isChecked); } } diff --git a/src/com/android/settings/fuelgauge/PowerBackgroundUsageDetail.java b/src/com/android/settings/fuelgauge/PowerBackgroundUsageDetail.java index 664d916cb40..28ae04fc721 100644 --- a/src/com/android/settings/fuelgauge/PowerBackgroundUsageDetail.java +++ b/src/com/android/settings/fuelgauge/PowerBackgroundUsageDetail.java @@ -28,7 +28,8 @@ import android.os.UserHandle; import android.text.TextUtils; import android.util.Log; import android.view.View; -import android.widget.Switch; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import androidx.annotation.VisibleForTesting; @@ -44,7 +45,6 @@ import com.android.settingslib.core.AbstractPreferenceController; import com.android.settingslib.widget.FooterPreference; import com.android.settingslib.widget.LayoutPreference; import com.android.settingslib.widget.MainSwitchPreference; -import com.android.settingslib.widget.OnMainSwitchChangeListener; import com.android.settingslib.widget.SelectorWithWidgetPreference; import java.util.ArrayList; @@ -54,7 +54,7 @@ import java.util.concurrent.Executors; /** Allow background usage fragment for each app */ public class PowerBackgroundUsageDetail extends DashboardFragment - implements SelectorWithWidgetPreference.OnClickListener, OnMainSwitchChangeListener { + implements SelectorWithWidgetPreference.OnClickListener, OnCheckedChangeListener { private static final String TAG = "PowerBackgroundUsageDetail"; public static final String EXTRA_UID = "extra_uid"; @@ -155,7 +155,7 @@ public class PowerBackgroundUsageDetail extends DashboardFragment } @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { mMainSwitchPreference.setChecked(isChecked); updateSelectorPreference(isChecked); } diff --git a/src/com/android/settings/gestures/PreventRingingSwitchPreferenceController.java b/src/com/android/settings/gestures/PreventRingingSwitchPreferenceController.java index 642623a80c3..9c0e0bffae7 100644 --- a/src/com/android/settings/gestures/PreventRingingSwitchPreferenceController.java +++ b/src/com/android/settings/gestures/PreventRingingSwitchPreferenceController.java @@ -18,7 +18,8 @@ package com.android.settings.gestures; import android.content.Context; import android.provider.Settings; -import android.widget.Switch; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import androidx.annotation.VisibleForTesting; import androidx.preference.Preference; @@ -28,10 +29,9 @@ import com.android.settings.R; import com.android.settings.core.PreferenceControllerMixin; import com.android.settingslib.core.AbstractPreferenceController; import com.android.settingslib.widget.MainSwitchPreference; -import com.android.settingslib.widget.OnMainSwitchChangeListener; public class PreventRingingSwitchPreferenceController extends AbstractPreferenceController - implements PreferenceControllerMixin, OnMainSwitchChangeListener { + implements PreferenceControllerMixin, OnCheckedChangeListener { private static final String KEY = "gesture_prevent_ringing_switch"; private final Context mContext; @@ -94,7 +94,7 @@ public class PreventRingingSwitchPreferenceController extends AbstractPreference } @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { final int preventRingingSetting = Settings.Secure.getInt(mContext.getContentResolver(), Settings.Secure.VOLUME_HUSH_GESTURE, Settings.Secure.VOLUME_HUSH_VIBRATE); final int newRingingSetting = preventRingingSetting == Settings.Secure.VOLUME_HUSH_OFF diff --git a/src/com/android/settings/inputmethod/SpellCheckersSettings.java b/src/com/android/settings/inputmethod/SpellCheckersSettings.java index a58d10ccf99..bcb527bb6d6 100644 --- a/src/com/android/settings/inputmethod/SpellCheckersSettings.java +++ b/src/com/android/settings/inputmethod/SpellCheckersSettings.java @@ -26,7 +26,8 @@ import android.util.Log; import android.view.textservice.SpellCheckerInfo; import android.view.textservice.SpellCheckerSubtype; import android.view.textservice.TextServicesManager; -import android.widget.Switch; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import androidx.appcompat.app.AlertDialog; import androidx.preference.Preference; @@ -37,10 +38,9 @@ import com.android.settings.R; import com.android.settings.SettingsActivity; import com.android.settings.SettingsPreferenceFragment; import com.android.settings.widget.SettingsMainSwitchBar; -import com.android.settingslib.widget.OnMainSwitchChangeListener; public class SpellCheckersSettings extends SettingsPreferenceFragment - implements OnMainSwitchChangeListener, OnPreferenceChangeListener { + implements OnCheckedChangeListener, OnPreferenceChangeListener { private static final String TAG = SpellCheckersSettings.class.getSimpleName(); private static final boolean DBG = false; @@ -105,7 +105,7 @@ public class SpellCheckersSettings extends SettingsPreferenceFragment } @Override - public void onSwitchChanged(final Switch switchView, final boolean isChecked) { + public void onCheckedChanged(final CompoundButton buttonView, final boolean isChecked) { Settings.Secure.putInt(getContentResolver(), Settings.Secure.SPELL_CHECKER_ENABLED, isChecked ? 1 : 0); updatePreferenceScreen(); diff --git a/src/com/android/settings/location/BluetoothScanningMainSwitchPreferenceController.java b/src/com/android/settings/location/BluetoothScanningMainSwitchPreferenceController.java index 78e31848acb..c975a410be1 100644 --- a/src/com/android/settings/location/BluetoothScanningMainSwitchPreferenceController.java +++ b/src/com/android/settings/location/BluetoothScanningMainSwitchPreferenceController.java @@ -16,22 +16,22 @@ package com.android.settings.location; import android.content.Context; -import android.provider.Settings; -import android.widget.Switch; import android.os.UserManager; +import android.provider.Settings; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import androidx.preference.PreferenceScreen; import com.android.settings.R; import com.android.settings.core.TogglePreferenceController; import com.android.settingslib.widget.MainSwitchPreference; -import com.android.settingslib.widget.OnMainSwitchChangeListener; /** * Preference controller for Bluetooth scanning main switch. */ public class BluetoothScanningMainSwitchPreferenceController extends TogglePreferenceController - implements OnMainSwitchChangeListener { + implements OnCheckedChangeListener { private static final String KEY_BLUETOOTH_SCANNING_SWITCH = "bluetooth_always_scanning_switch"; private final UserManager mUserManager; @@ -78,7 +78,7 @@ public class BluetoothScanningMainSwitchPreferenceController extends TogglePrefe } @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { if (isChecked != isChecked()) { setChecked(isChecked); } diff --git a/src/com/android/settings/location/LocationSwitchBarController.java b/src/com/android/settings/location/LocationSwitchBarController.java index b150b1d93c0..1e415d68f8d 100644 --- a/src/com/android/settings/location/LocationSwitchBarController.java +++ b/src/com/android/settings/location/LocationSwitchBarController.java @@ -16,7 +16,8 @@ package com.android.settings.location; import android.content.Context; import android.os.UserHandle; import android.os.UserManager; -import android.widget.Switch; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import com.android.settings.widget.SettingsMainSwitchBar; import com.android.settingslib.RestrictedLockUtils; @@ -24,12 +25,11 @@ import com.android.settingslib.core.lifecycle.Lifecycle; import com.android.settingslib.core.lifecycle.LifecycleObserver; import com.android.settingslib.core.lifecycle.events.OnStart; import com.android.settingslib.core.lifecycle.events.OnStop; -import com.android.settingslib.widget.OnMainSwitchChangeListener; /** * The switch controller for the location. */ -public class LocationSwitchBarController implements OnMainSwitchChangeListener, +public class LocationSwitchBarController implements OnCheckedChangeListener, LocationEnabler.LocationModeChangeListener, LifecycleObserver, OnStart, OnStop { private final SettingsMainSwitchBar mSwitchBar; @@ -100,7 +100,7 @@ public class LocationSwitchBarController implements OnMainSwitchChangeListener, * Listens to the state change of the location primary switch. */ @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { mLocationEnabler.setLocationEnabled(isChecked); } } diff --git a/src/com/android/settings/location/WifiScanningMainSwitchPreferenceController.java b/src/com/android/settings/location/WifiScanningMainSwitchPreferenceController.java index e22b0a08026..9f19a703e65 100644 --- a/src/com/android/settings/location/WifiScanningMainSwitchPreferenceController.java +++ b/src/com/android/settings/location/WifiScanningMainSwitchPreferenceController.java @@ -17,21 +17,21 @@ package com.android.settings.location; import android.content.Context; import android.net.wifi.WifiManager; -import android.widget.Switch; import android.os.UserManager; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import androidx.preference.PreferenceScreen; import com.android.settings.R; import com.android.settings.core.TogglePreferenceController; import com.android.settingslib.widget.MainSwitchPreference; -import com.android.settingslib.widget.OnMainSwitchChangeListener; /** * Preference controller for Wi-Fi scanning main switch. */ public class WifiScanningMainSwitchPreferenceController extends TogglePreferenceController - implements OnMainSwitchChangeListener { + implements OnCheckedChangeListener { private static final String KEY_WIFI_SCANNING_SWITCH = "wifi_always_scanning_switch"; private final WifiManager mWifiManager; @@ -79,7 +79,7 @@ public class WifiScanningMainSwitchPreferenceController extends TogglePreference } @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { if (isChecked != isChecked()) { setChecked(isChecked); } diff --git a/src/com/android/settings/network/telephony/MobileNetworkSwitchController.java b/src/com/android/settings/network/telephony/MobileNetworkSwitchController.java index 453941d706c..20a3d8921f7 100644 --- a/src/com/android/settings/network/telephony/MobileNetworkSwitchController.java +++ b/src/com/android/settings/network/telephony/MobileNetworkSwitchController.java @@ -86,7 +86,7 @@ public class MobileNetworkSwitchController extends BasePreferenceController impl super.displayPreference(screen); mSwitchBar = (SettingsMainSwitchPreference) screen.findPreference(mPreferenceKey); - mSwitchBar.setOnBeforeCheckedChangeListener((toggleSwitch, isChecked) -> { + mSwitchBar.setOnBeforeCheckedChangeListener((isChecked) -> { // TODO b/135222940: re-evaluate whether to use // mSubscriptionManager#isSubscriptionEnabled if (mSubscriptionManager.isActiveSubscriptionId(mSubId) != isChecked) { diff --git a/src/com/android/settings/nfc/NfcPreferenceController.java b/src/com/android/settings/nfc/NfcPreferenceController.java index 2ba00c6670a..4d750042919 100644 --- a/src/com/android/settings/nfc/NfcPreferenceController.java +++ b/src/com/android/settings/nfc/NfcPreferenceController.java @@ -22,7 +22,8 @@ import android.content.IntentFilter; import android.net.Uri; import android.nfc.NfcAdapter; import android.util.Log; -import android.widget.Switch; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import androidx.preference.PreferenceScreen; @@ -33,12 +34,11 @@ import com.android.settingslib.core.lifecycle.LifecycleObserver; import com.android.settingslib.core.lifecycle.events.OnPause; import com.android.settingslib.core.lifecycle.events.OnResume; import com.android.settingslib.widget.MainSwitchPreference; -import com.android.settingslib.widget.OnMainSwitchChangeListener; import java.io.IOException; public class NfcPreferenceController extends TogglePreferenceController - implements LifecycleObserver, OnResume, OnPause, OnMainSwitchChangeListener { + implements LifecycleObserver, OnResume, OnPause, OnCheckedChangeListener { public static final String KEY_TOGGLE_NFC = "toggle_nfc"; private final NfcAdapter mNfcAdapter; @@ -64,7 +64,7 @@ public class NfcPreferenceController extends TogglePreferenceController } @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { if (isChecked != mNfcAdapter.isEnabled()) { setChecked(isChecked); } diff --git a/src/com/android/settings/notification/app/BlockPreferenceController.java b/src/com/android/settings/notification/app/BlockPreferenceController.java index f4e213298a7..a3d0a730ced 100644 --- a/src/com/android/settings/notification/app/BlockPreferenceController.java +++ b/src/com/android/settings/notification/app/BlockPreferenceController.java @@ -23,7 +23,8 @@ import static android.app.NotificationManager.IMPORTANCE_UNSPECIFIED; import android.app.NotificationChannel; import android.app.NotificationManager; import android.content.Context; -import android.widget.Switch; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import androidx.preference.Preference; @@ -31,10 +32,9 @@ import com.android.settings.R; import com.android.settings.core.PreferenceControllerMixin; import com.android.settings.notification.NotificationBackend; import com.android.settings.widget.SettingsMainSwitchPreference; -import com.android.settingslib.widget.OnMainSwitchChangeListener; public class BlockPreferenceController extends NotificationPreferenceController - implements PreferenceControllerMixin, OnMainSwitchChangeListener { + implements PreferenceControllerMixin, OnCheckedChangeListener { private static final String KEY_BLOCK = "block"; private NotificationSettings.DependentFieldListener mDependentFieldListener; @@ -104,7 +104,7 @@ public class BlockPreferenceController extends NotificationPreferenceController } @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { boolean blocked = !isChecked; if (mChannel != null) { final int originalImportance = mChannel.getImportance(); diff --git a/src/com/android/settings/notification/history/NotificationHistoryActivity.java b/src/com/android/settings/notification/history/NotificationHistoryActivity.java index 4808773f52e..4616934e684 100644 --- a/src/com/android/settings/notification/history/NotificationHistoryActivity.java +++ b/src/com/android/settings/notification/history/NotificationHistoryActivity.java @@ -47,6 +47,7 @@ import android.view.LayoutInflater; import android.view.View; import android.view.ViewGroup; import android.view.ViewOutlineProvider; +import android.widget.CompoundButton.OnCheckedChangeListener; import android.widget.ImageView; import android.widget.TextView; @@ -64,7 +65,6 @@ import com.android.settingslib.collapsingtoolbar.CollapsingToolbarBaseActivity; import com.android.settingslib.utils.StringUtil; import com.android.settingslib.utils.ThreadUtils; import com.android.settingslib.widget.MainSwitchBar; -import com.android.settingslib.widget.OnMainSwitchChangeListener; import java.util.ArrayList; import java.util.Arrays; @@ -355,7 +355,7 @@ public class NotificationHistoryActivity extends CollapsingToolbarBaseActivity { mHistoryEmpty.setVisibility(View.GONE); } - private final OnMainSwitchChangeListener mOnSwitchClickListener = + private final OnCheckedChangeListener mOnSwitchClickListener = (switchView, isChecked) -> { int oldState = 0; try { diff --git a/src/com/android/settings/notification/zen/ZenAutomaticRuleSwitchPreferenceController.java b/src/com/android/settings/notification/zen/ZenAutomaticRuleSwitchPreferenceController.java index fa36dde5171..4d2a6a58d6c 100644 --- a/src/com/android/settings/notification/zen/ZenAutomaticRuleSwitchPreferenceController.java +++ b/src/com/android/settings/notification/zen/ZenAutomaticRuleSwitchPreferenceController.java @@ -18,7 +18,8 @@ package com.android.settings.notification.zen; import android.app.AutomaticZenRule; import android.content.Context; -import android.widget.Switch; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import androidx.fragment.app.Fragment; import androidx.preference.Preference; @@ -27,11 +28,10 @@ import androidx.preference.PreferenceScreen; import com.android.settings.R; import com.android.settingslib.core.lifecycle.Lifecycle; import com.android.settingslib.widget.MainSwitchPreference; -import com.android.settingslib.widget.OnMainSwitchChangeListener; public class ZenAutomaticRuleSwitchPreferenceController extends AbstractZenModeAutomaticRulePreferenceController implements - OnMainSwitchChangeListener { + OnCheckedChangeListener { private static final String KEY = "zen_automatic_rule_switch"; private AutomaticZenRule mRule; @@ -86,7 +86,7 @@ public class ZenAutomaticRuleSwitchPreferenceController extends } @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { final boolean enabled = isChecked; if (enabled == mRule.isEnabled()) return; mRule.setEnabled(enabled); diff --git a/src/com/android/settings/print/PrintServiceSettingsFragment.java b/src/com/android/settings/print/PrintServiceSettingsFragment.java index c5316e6e524..e144ffdd3c9 100644 --- a/src/com/android/settings/print/PrintServiceSettingsFragment.java +++ b/src/com/android/settings/print/PrintServiceSettingsFragment.java @@ -43,12 +43,13 @@ import android.view.View; import android.view.View.OnClickListener; import android.view.ViewGroup; import android.view.accessibility.AccessibilityManager; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import android.widget.Filter; import android.widget.Filterable; import android.widget.ImageView; import android.widget.LinearLayout; import android.widget.SearchView; -import android.widget.Switch; import android.widget.TextView; import androidx.annotation.NonNull; @@ -61,7 +62,6 @@ import com.android.settings.R; import com.android.settings.SettingsActivity; import com.android.settings.SettingsPreferenceFragment; import com.android.settings.widget.SettingsMainSwitchBar; -import com.android.settingslib.widget.OnMainSwitchChangeListener; import java.util.ArrayList; import java.util.LinkedHashMap; @@ -72,7 +72,7 @@ import java.util.Map; * Fragment with print service settings. */ public class PrintServiceSettingsFragment extends SettingsPreferenceFragment - implements OnMainSwitchChangeListener, + implements OnCheckedChangeListener, LoaderManager.LoaderCallbacks> { private static final String LOG_TAG = "PrintServiceSettings"; @@ -240,7 +240,7 @@ public class PrintServiceSettingsFragment extends SettingsPreferenceFragment mSwitchBar.addOnSwitchChangeListener(this); mSwitchBar.show(); - mSwitchBar.setOnBeforeCheckedChangeListener((toggleSwitch, checked) -> { + mSwitchBar.setOnBeforeCheckedChangeListener((checked) -> { onPreferenceToggled(mPreferenceKey, checked); return false; }); @@ -250,7 +250,7 @@ public class PrintServiceSettingsFragment extends SettingsPreferenceFragment @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { updateEmptyView(); } diff --git a/src/com/android/settings/security/ScreenPinningSettings.java b/src/com/android/settings/security/ScreenPinningSettings.java index 20b640315be..99d6492d1b9 100644 --- a/src/com/android/settings/security/ScreenPinningSettings.java +++ b/src/com/android/settings/security/ScreenPinningSettings.java @@ -27,7 +27,8 @@ import android.os.Bundle; import android.os.UserHandle; import android.os.UserManager; import android.provider.Settings; -import android.widget.Switch; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import androidx.appcompat.app.AlertDialog; import androidx.preference.Preference; @@ -45,13 +46,12 @@ import com.android.settings.search.BaseSearchIndexProvider; import com.android.settings.widget.SettingsMainSwitchBar; import com.android.settingslib.search.SearchIndexable; import com.android.settingslib.widget.FooterPreference; -import com.android.settingslib.widget.OnMainSwitchChangeListener; /** * Screen pinning settings. */ @SearchIndexable public class ScreenPinningSettings extends SettingsPreferenceFragment - implements OnMainSwitchChangeListener, DialogInterface.OnClickListener { + implements OnCheckedChangeListener, DialogInterface.OnClickListener { private static final String KEY_USE_SCREEN_LOCK = "use_screen_lock"; private static final String KEY_FOOTER = "screen_pinning_settings_screen_footer"; @@ -198,7 +198,7 @@ public class ScreenPinningSettings extends SettingsPreferenceFragment * Listens to the state change of the overall lock-to-app switch. */ @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { if (isChecked) { new AlertDialog.Builder(getContext()) .setMessage(R.string.screen_pinning_dialog_message) diff --git a/src/com/android/settings/widget/MainSwitchBarController.java b/src/com/android/settings/widget/MainSwitchBarController.java index 8aa5c9b5db9..455da94c43b 100644 --- a/src/com/android/settings/widget/MainSwitchBarController.java +++ b/src/com/android/settings/widget/MainSwitchBarController.java @@ -16,16 +16,16 @@ package com.android.settings.widget; -import android.widget.Switch; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import com.android.settingslib.RestrictedLockUtils; -import com.android.settingslib.widget.OnMainSwitchChangeListener; /** * The switch controller that is used to update the switch widget in the SettingsMainSwitchBar. */ public class MainSwitchBarController extends SwitchWidgetController implements - OnMainSwitchChangeListener { + OnCheckedChangeListener { private final SettingsMainSwitchBar mMainSwitch; @@ -74,7 +74,7 @@ public class MainSwitchBarController extends SwitchWidgetController implements } @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { if (mListener != null) { mListener.onSwitchToggled(isChecked); } diff --git a/src/com/android/settings/widget/SettingsMainSwitchBar.java b/src/com/android/settings/widget/SettingsMainSwitchBar.java index ac2c136e8a1..e8c6fc9b0b9 100644 --- a/src/com/android/settings/widget/SettingsMainSwitchBar.java +++ b/src/com/android/settings/widget/SettingsMainSwitchBar.java @@ -21,7 +21,6 @@ import static com.android.settingslib.RestrictedLockUtils.EnforcedAdmin; import android.content.Context; import android.util.AttributeSet; import android.view.View; -import android.widget.Switch; import com.android.settings.overlay.FeatureFactory; import com.android.settingslib.RestrictedLockUtils; @@ -39,10 +38,9 @@ public class SettingsMainSwitchBar extends MainSwitchBar { public interface OnBeforeCheckedChangeListener { /** - * @param switchView The Switch view whose state has changed. - * @param isChecked The new checked state of switchView. + * @param isChecked The new checked state of switchView. */ - boolean onBeforeCheckedChanged(Switch switchView, boolean isChecked); + boolean onBeforeCheckedChanged(boolean isChecked); } private EnforcedAdmin mEnforcedAdmin; @@ -117,7 +115,7 @@ public class SettingsMainSwitchBar extends MainSwitchBar { @Override public void setChecked(boolean checked) { if (mOnBeforeListener != null - && mOnBeforeListener.onBeforeCheckedChanged(mSwitch, checked)) { + && mOnBeforeListener.onBeforeCheckedChanged(checked)) { return; } super.setChecked(checked); diff --git a/src/com/android/settings/widget/SettingsMainSwitchPreference.java b/src/com/android/settings/widget/SettingsMainSwitchPreference.java index ba17c856999..fc43e585106 100644 --- a/src/com/android/settings/widget/SettingsMainSwitchPreference.java +++ b/src/com/android/settings/widget/SettingsMainSwitchPreference.java @@ -20,7 +20,8 @@ import static com.android.settingslib.RestrictedLockUtils.EnforcedAdmin; import android.content.Context; import android.util.AttributeSet; -import android.widget.Switch; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import androidx.preference.PreferenceViewHolder; import androidx.preference.TwoStatePreference; @@ -29,7 +30,6 @@ import com.android.settings.R; import com.android.settings.widget.SettingsMainSwitchBar.OnBeforeCheckedChangeListener; import com.android.settingslib.RestrictedPreferenceHelper; import com.android.settingslib.core.instrumentation.SettingsJankMonitor; -import com.android.settingslib.widget.OnMainSwitchChangeListener; import java.util.ArrayList; import java.util.List; @@ -40,11 +40,11 @@ import java.util.List; * to enable or disable the preferences on the page. */ public class SettingsMainSwitchPreference extends TwoStatePreference implements - OnMainSwitchChangeListener { + OnCheckedChangeListener { private final List mBeforeCheckedChangeListeners = new ArrayList<>(); - private final List mSwitchChangeListeners = new ArrayList<>(); + private final List mSwitchChangeListeners = new ArrayList<>(); private SettingsMainSwitchBar mMainSwitchBar; private EnforcedAdmin mEnforcedAdmin; @@ -127,9 +127,9 @@ public class SettingsMainSwitchPreference extends TwoStatePreference implements } @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { super.setChecked(isChecked); - SettingsJankMonitor.detectToggleJank(getKey(), switchView); + SettingsJankMonitor.detectToggleJank(getKey(), buttonView); } /** @@ -197,7 +197,7 @@ public class SettingsMainSwitchPreference extends TwoStatePreference implements /** * Adds a listener for switch changes */ - public void addOnSwitchChangeListener(OnMainSwitchChangeListener listener) { + public void addOnSwitchChangeListener(OnCheckedChangeListener listener) { if (!mSwitchChangeListeners.contains(listener)) { mSwitchChangeListeners.add(listener); } @@ -209,7 +209,7 @@ public class SettingsMainSwitchPreference extends TwoStatePreference implements /** * Remove a listener for switch changes */ - public void removeOnSwitchChangeListener(OnMainSwitchChangeListener listener) { + public void removeOnSwitchChangeListener(OnCheckedChangeListener listener) { mSwitchChangeListeners.remove(listener); if (mMainSwitchBar != null) { mMainSwitchBar.removeOnSwitchChangeListener(listener); @@ -239,7 +239,7 @@ public class SettingsMainSwitchPreference extends TwoStatePreference implements for (OnBeforeCheckedChangeListener listener : mBeforeCheckedChangeListeners) { mMainSwitchBar.setOnBeforeCheckedChangeListener(listener); } - for (OnMainSwitchChangeListener listener : mSwitchChangeListeners) { + for (OnCheckedChangeListener listener : mSwitchChangeListeners) { mMainSwitchBar.addOnSwitchChangeListener(listener); } } diff --git a/src/com/android/settings/widget/SettingsMainSwitchPreferenceController.java b/src/com/android/settings/widget/SettingsMainSwitchPreferenceController.java index 5fa9259ab9e..60e3388b264 100644 --- a/src/com/android/settings/widget/SettingsMainSwitchPreferenceController.java +++ b/src/com/android/settings/widget/SettingsMainSwitchPreferenceController.java @@ -17,20 +17,20 @@ package com.android.settings.widget; import android.content.Context; -import android.widget.Switch; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import androidx.preference.Preference; import androidx.preference.PreferenceScreen; import com.android.settings.core.TogglePreferenceController; import com.android.settingslib.widget.MainSwitchPreference; -import com.android.settingslib.widget.OnMainSwitchChangeListener; /** * Preference controller for MainSwitchPreference. */ public abstract class SettingsMainSwitchPreferenceController extends - TogglePreferenceController implements OnMainSwitchChangeListener { + TogglePreferenceController implements OnCheckedChangeListener { protected MainSwitchPreference mSwitchPreference; @@ -49,7 +49,7 @@ public abstract class SettingsMainSwitchPreferenceController extends } @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { mSwitchPreference.setChecked(isChecked); setChecked(isChecked); } diff --git a/src/com/android/settings/wifi/calling/WifiCallingSettingsForSub.java b/src/com/android/settings/wifi/calling/WifiCallingSettingsForSub.java index 650536e52e6..739f8b25b23 100644 --- a/src/com/android/settings/wifi/calling/WifiCallingSettingsForSub.java +++ b/src/com/android/settings/wifi/calling/WifiCallingSettingsForSub.java @@ -38,7 +38,8 @@ import android.util.Log; import android.view.LayoutInflater; import android.view.View; import android.view.ViewGroup; -import android.widget.Switch; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import androidx.appcompat.app.AlertDialog; import androidx.preference.Preference; @@ -55,7 +56,6 @@ import com.android.settings.Utils; import com.android.settings.core.SubSettingLauncher; import com.android.settings.network.ims.WifiCallingQueryImsState; import com.android.settings.widget.SettingsMainSwitchPreference; -import com.android.settingslib.widget.OnMainSwitchChangeListener; import java.util.List; @@ -64,7 +64,7 @@ import java.util.List; * The preference screen lets you enable/disable Wi-Fi Calling and change Wi-Fi Calling mode. */ public class WifiCallingSettingsForSub extends SettingsPreferenceFragment - implements OnMainSwitchChangeListener, + implements OnCheckedChangeListener, Preference.OnPreferenceChangeListener { private static final String TAG = "WifiCallingForSub"; @@ -463,7 +463,7 @@ public class WifiCallingSettingsForSub extends SettingsPreferenceFragment * Listens to the state change of the switch. */ @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { Log.d(TAG, "onSwitchChanged(" + isChecked + ")"); if (!isChecked) { diff --git a/src/com/android/settings/wifi/tether/WifiTetherSwitchBarController.java b/src/com/android/settings/wifi/tether/WifiTetherSwitchBarController.java index 75a37898035..9febba3b7d0 100644 --- a/src/com/android/settings/wifi/tether/WifiTetherSwitchBarController.java +++ b/src/com/android/settings/wifi/tether/WifiTetherSwitchBarController.java @@ -32,7 +32,8 @@ import android.net.wifi.WifiManager; import android.os.Handler; import android.os.Looper; import android.util.Log; -import android.widget.Switch; +import android.widget.CompoundButton; +import android.widget.CompoundButton.OnCheckedChangeListener; import androidx.annotation.VisibleForTesting; @@ -41,13 +42,12 @@ import com.android.settings.widget.SettingsMainSwitchBar; import com.android.settingslib.core.lifecycle.LifecycleObserver; import com.android.settingslib.core.lifecycle.events.OnStart; import com.android.settingslib.core.lifecycle.events.OnStop; -import com.android.settingslib.widget.OnMainSwitchChangeListener; /** * Controller for logic pertaining to switch Wi-Fi tethering. */ public class WifiTetherSwitchBarController implements - LifecycleObserver, OnStart, OnStop, DataSaverBackend.Listener, OnMainSwitchChangeListener { + LifecycleObserver, OnStart, OnStop, DataSaverBackend.Listener, OnCheckedChangeListener { private static final String TAG = "WifiTetherSBC"; private static final IntentFilter WIFI_INTENT_FILTER; @@ -101,9 +101,9 @@ public class WifiTetherSwitchBarController implements } @Override - public void onSwitchChanged(Switch switchView, boolean isChecked) { + public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) { // Filter out unnecessary callbacks when switch is disabled. - if (!switchView.isEnabled()) return; + if (!buttonView.isEnabled()) return; if (isChecked) { startTether(); diff --git a/tests/robotests/src/com/android/settings/accessibility/CaptioningTogglePreferenceControllerTest.java b/tests/robotests/src/com/android/settings/accessibility/CaptioningTogglePreferenceControllerTest.java index cdc17410493..e0a04bce8ca 100644 --- a/tests/robotests/src/com/android/settings/accessibility/CaptioningTogglePreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/accessibility/CaptioningTogglePreferenceControllerTest.java @@ -142,7 +142,7 @@ public class CaptioningTogglePreferenceControllerTest { public void onSwitchChanged_switchChecked_shouldSetCaptionEnabled() { mController.displayPreference(mScreen); - mController.onSwitchChanged(/* switchView= */ null, /* isChecked= */ true); + mController.onCheckedChanged(/* buttonView= */ null, /* isChecked= */ true); assertThat(isCaptionEnabled()).isTrue(); } @@ -151,7 +151,7 @@ public class CaptioningTogglePreferenceControllerTest { public void onSwitchChanged_switchUnchecked_shouldSetCaptionDisabled() { mController.displayPreference(mScreen); - mController.onSwitchChanged(/* switchView= */ null, /* isChecked= */ false); + mController.onCheckedChanged(/* buttonView= */ null, /* isChecked= */ false); assertThat(isCaptionEnabled()).isFalse(); } diff --git a/tests/robotests/src/com/android/settings/accounts/WorkModePreferenceControllerTest.java b/tests/robotests/src/com/android/settings/accounts/WorkModePreferenceControllerTest.java index d6be73d5481..e53ed105858 100644 --- a/tests/robotests/src/com/android/settings/accounts/WorkModePreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/accounts/WorkModePreferenceControllerTest.java @@ -115,11 +115,11 @@ public class WorkModePreferenceControllerTest { @Test public void onPreferenceChange_shouldRequestQuietModeEnabled() { - mController.onSwitchChanged(mSwitch, true); + mController.onCheckedChanged(mSwitch, true); verify(mUserManager).requestQuietModeEnabled(false, mManagedUser); - mController.onSwitchChanged(mSwitch, false); + mController.onCheckedChanged(mSwitch, false); verify(mUserManager).requestQuietModeEnabled(true, mManagedUser); } diff --git a/tests/robotests/src/com/android/settings/deletionhelper/AutomaticStorageManagerSwitchBarControllerTest.java b/tests/robotests/src/com/android/settings/deletionhelper/AutomaticStorageManagerSwitchBarControllerTest.java index 17283cfaf38..9841a0f8d19 100644 --- a/tests/robotests/src/com/android/settings/deletionhelper/AutomaticStorageManagerSwitchBarControllerTest.java +++ b/tests/robotests/src/com/android/settings/deletionhelper/AutomaticStorageManagerSwitchBarControllerTest.java @@ -80,7 +80,7 @@ public class AutomaticStorageManagerSwitchBarControllerTest { @Test public void onSwitchChanged_false_recordsAMetric() { - mController.onSwitchChanged(null, false); + mController.onCheckedChanged(null, false); verify(mMetricsFeatureProvider) .action( @@ -91,7 +91,7 @@ public class AutomaticStorageManagerSwitchBarControllerTest { @Test public void onSwitchChanged_true_recordsAMetric() { - mController.onSwitchChanged(null, true); + mController.onCheckedChanged(null, true); verify(mMetricsFeatureProvider) .action( @@ -102,7 +102,7 @@ public class AutomaticStorageManagerSwitchBarControllerTest { @Test public void onSwitchChanged_showWarningFragmentIfNotEnabledByDefault() { - mController.onSwitchChanged(null, true); + mController.onCheckedChanged(null, true); verify(mFragmentManager.beginTransaction()) .add(any(Fragment.class), eq(ActivationWarningFragment.TAG)); @@ -112,7 +112,7 @@ public class AutomaticStorageManagerSwitchBarControllerTest { public void onSwitchChange_doNotShowWarningFragmentIfEnabledByDefault() { SystemProperties.set("ro.storage_manager.enabled", "true"); - mController.onSwitchChanged(null, true); + mController.onCheckedChanged(null, true); verify(mFragmentManager.beginTransaction(), never()) .add(any(Fragment.class), eq(ActivationWarningFragment.TAG)); diff --git a/tests/robotests/src/com/android/settings/development/DevelopmentSettingsDashboardFragmentTest.java b/tests/robotests/src/com/android/settings/development/DevelopmentSettingsDashboardFragmentTest.java index 3189b47ab80..37a4aeab572 100644 --- a/tests/robotests/src/com/android/settings/development/DevelopmentSettingsDashboardFragmentTest.java +++ b/tests/robotests/src/com/android/settings/development/DevelopmentSettingsDashboardFragmentTest.java @@ -160,7 +160,7 @@ public class DevelopmentSettingsDashboardFragmentTest { Settings.Global.putInt(mContext.getContentResolver(), Settings.Global.DEVELOPMENT_SETTINGS_ENABLED, 0); - mDashboard.onSwitchChanged(null, false /* isChecked */); + mDashboard.onCheckedChanged(null, false /* isChecked */); assertThat(ShadowEnableDevelopmentSettingWarningDialog.mShown).isFalse(); } @@ -172,7 +172,7 @@ public class DevelopmentSettingsDashboardFragmentTest { Settings.Global.putInt(mContext.getContentResolver(), Settings.Global.DEVELOPMENT_SETTINGS_ENABLED, 0); - mDashboard.onSwitchChanged(null, true /* isChecked */); + mDashboard.onCheckedChanged(null, true /* isChecked */); assertThat(ShadowEnableDevelopmentSettingWarningDialog.mShown).isTrue(); } @@ -184,7 +184,7 @@ public class DevelopmentSettingsDashboardFragmentTest { Settings.Global.putInt(mContext.getContentResolver(), Settings.Global.DEVELOPMENT_SETTINGS_ENABLED, 1); - mDashboard.onSwitchChanged(null, false /* isChecked */); + mDashboard.onCheckedChanged(null, false /* isChecked */); assertThat(ShadowEnableDevelopmentSettingWarningDialog.mShown).isFalse(); assertThat(DevelopmentSettingsEnabler.isDevelopmentSettingsEnabled(mContext)).isFalse(); @@ -203,7 +203,7 @@ public class DevelopmentSettingsDashboardFragmentTest { Settings.Global.putInt(mContext.getContentResolver(), Settings.Global.DEVELOPMENT_SETTINGS_ENABLED, 1); - mDashboard.onSwitchChanged(null, false /* isChecked */); + mDashboard.onCheckedChanged(null, false /* isChecked */); AlertDialog dialog = ShadowAlertDialogCompat.getLatestAlertDialog(); assertThat(dialog).isNotNull(); diff --git a/tests/robotests/src/com/android/settings/development/DevelopmentSwitchBarControllerTest.java b/tests/robotests/src/com/android/settings/development/DevelopmentSwitchBarControllerTest.java index f776536d2d1..34d22e09649 100644 --- a/tests/robotests/src/com/android/settings/development/DevelopmentSwitchBarControllerTest.java +++ b/tests/robotests/src/com/android/settings/development/DevelopmentSwitchBarControllerTest.java @@ -24,6 +24,7 @@ import static com.google.common.truth.Truth.assertThat; import static org.mockito.Mockito.when; import android.content.Context; +import android.widget.CompoundButton.OnCheckedChangeListener; import androidx.lifecycle.LifecycleOwner; @@ -31,7 +32,6 @@ import com.android.settings.testutils.shadow.ShadowUserManager; import com.android.settings.testutils.shadow.ShadowUtils; import com.android.settings.widget.SettingsMainSwitchBar; import com.android.settingslib.core.lifecycle.Lifecycle; -import com.android.settingslib.widget.OnMainSwitchChangeListener; import org.junit.After; import org.junit.Before; @@ -78,7 +78,7 @@ public class DevelopmentSwitchBarControllerTest { ShadowUtils.setIsUserAMonkey(true); new DevelopmentSwitchBarController(mSettings, mSwitchBar, true /* isAvailable */, mLifecycle); - final List listeners = + final List listeners = ReflectionHelpers.getField(mSwitchBar, "mSwitchChangeListeners"); mLifecycle.handleLifecycleEvent(ON_START); @@ -93,7 +93,7 @@ public class DevelopmentSwitchBarControllerTest { ShadowUtils.setIsUserAMonkey(false); new DevelopmentSwitchBarController(mSettings, mSwitchBar, true /* isAvailable */, mLifecycle); - final List listeners = + final List listeners = ReflectionHelpers.getField(mSwitchBar, "mSwitchChangeListeners"); mLifecycle.handleLifecycleEvent(ON_START); @@ -109,7 +109,7 @@ public class DevelopmentSwitchBarControllerTest { ShadowUtils.setIsUserAMonkey(false); new DevelopmentSwitchBarController(mSettings, mSwitchBar, true /* isAvailable */, mLifecycle); - final List listeners = + final List listeners = ReflectionHelpers.getField(mSwitchBar, "mSwitchChangeListeners"); mLifecycle.handleLifecycleEvent(ON_START); diff --git a/tests/robotests/src/com/android/settings/display/NightDisplayActivationPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/display/NightDisplayActivationPreferenceControllerTest.java index d084c512938..5c38411855c 100644 --- a/tests/robotests/src/com/android/settings/display/NightDisplayActivationPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/display/NightDisplayActivationPreferenceControllerTest.java @@ -104,7 +104,7 @@ public class NightDisplayActivationPreferenceControllerTest { final NightDisplayActivationPreferenceController controller = new NightDisplayActivationPreferenceController(mContext, "night_display_activated"); - controller.onSwitchChanged(null, true); + controller.onCheckedChanged(null, true); assertThat(mColorDisplayManager.isNightDisplayActivated()).isEqualTo(true); } @@ -115,7 +115,7 @@ public class NightDisplayActivationPreferenceControllerTest { final NightDisplayActivationPreferenceController controller = new NightDisplayActivationPreferenceController(mContext, "night_display_activated"); - controller.onSwitchChanged(null, false); + controller.onCheckedChanged(null, false); assertThat(mColorDisplayManager.isNightDisplayActivated()).isEqualTo(false); } diff --git a/tests/robotests/src/com/android/settings/dream/DreamMainSwitchPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/dream/DreamMainSwitchPreferenceControllerTest.java index fb55164b073..924c63ac0f8 100644 --- a/tests/robotests/src/com/android/settings/dream/DreamMainSwitchPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/dream/DreamMainSwitchPreferenceControllerTest.java @@ -28,7 +28,6 @@ import androidx.test.core.app.ApplicationProvider; import com.android.settingslib.dream.DreamBackend; import com.android.settingslib.widget.MainSwitchPreference; -import com.android.settingslib.widget.OnMainSwitchChangeListener; import org.junit.After; import org.junit.Before; @@ -53,8 +52,6 @@ public class DreamMainSwitchPreferenceControllerTest { private DreamMainSwitchPreferenceController mController; private MainSwitchPreference mPreference; private DreamBackend mBackend; - @Mock - private OnMainSwitchChangeListener mChangeListener; private ShadowContentResolver mShadowContentResolver; @Before diff --git a/tests/robotests/src/com/android/settings/fuelgauge/PowerBackgroundUsageDetailTest.java b/tests/robotests/src/com/android/settings/fuelgauge/PowerBackgroundUsageDetailTest.java index e50edecec09..3b80efbdecf 100644 --- a/tests/robotests/src/com/android/settings/fuelgauge/PowerBackgroundUsageDetailTest.java +++ b/tests/robotests/src/com/android/settings/fuelgauge/PowerBackgroundUsageDetailTest.java @@ -39,7 +39,7 @@ import android.content.pm.PackageManager; import android.graphics.drawable.Drawable; import android.os.Bundle; import android.os.UserHandle; -import android.widget.Switch; +import android.widget.CompoundButton; import androidx.fragment.app.FragmentActivity; import androidx.loader.app.LoaderManager; @@ -109,7 +109,7 @@ public class PowerBackgroundUsageDetailTest { @Mock private BackupManager mBackupManager; @Mock private PackageManager mPackageManager; @Mock private AppOpsManager mAppOpsManager; - @Mock private Switch mMockSwitch; + @Mock private CompoundButton mMockSwitch; @Before public void setUp() { @@ -259,7 +259,7 @@ public class PowerBackgroundUsageDetailTest { final int optimizedMode = BatteryOptimizeUtils.MODE_OPTIMIZED; mFragment.mOptimizationMode = optimizedMode; - mFragment.onSwitchChanged(mMockSwitch, /* isChecked= */ false); + mFragment.onCheckedChanged(mMockSwitch, /* isChecked= */ false); verify(mOptimizePreference).setEnabled(false); verify(mUnrestrictedPreference).setEnabled(false); @@ -275,7 +275,7 @@ public class PowerBackgroundUsageDetailTest { final int optimizedMode = BatteryOptimizeUtils.MODE_OPTIMIZED; mFragment.mOptimizationMode = restrictedMode; - mFragment.onSwitchChanged(mMockSwitch, /* isChecked= */ true); + mFragment.onCheckedChanged(mMockSwitch, /* isChecked= */ true); verify(mOptimizePreference).setEnabled(true); verify(mUnrestrictedPreference).setEnabled(true); diff --git a/tests/robotests/src/com/android/settings/gestures/PreventRingingSwitchPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/gestures/PreventRingingSwitchPreferenceControllerTest.java index 5f25759ba93..67618c8ab3b 100644 --- a/tests/robotests/src/com/android/settings/gestures/PreventRingingSwitchPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/gestures/PreventRingingSwitchPreferenceControllerTest.java @@ -116,7 +116,7 @@ public class PreventRingingSwitchPreferenceControllerTest { Settings.Secure.putInt(mContext.getContentResolver(), Settings.Secure.VOLUME_HUSH_GESTURE, VOLUME_HUSH_OFF); - mController.onSwitchChanged(null, true); + mController.onCheckedChanged(null, true); assertThat(Settings.Secure.getInt(mContext.getContentResolver(), Settings.Secure.VOLUME_HUSH_GESTURE, UNKNOWN)).isEqualTo(VOLUME_HUSH_VIBRATE); @@ -127,7 +127,7 @@ public class PreventRingingSwitchPreferenceControllerTest { Settings.Secure.putInt(mContext.getContentResolver(), Settings.Secure.VOLUME_HUSH_GESTURE, VOLUME_HUSH_MUTE); - mController.onSwitchChanged(null, false); + mController.onCheckedChanged(null, false); assertThat(Settings.Secure.getInt(mContext.getContentResolver(), Settings.Secure.VOLUME_HUSH_GESTURE, UNKNOWN)).isEqualTo(VOLUME_HUSH_OFF); @@ -139,7 +139,7 @@ public class PreventRingingSwitchPreferenceControllerTest { Settings.Secure.putInt(mContext.getContentResolver(), Settings.Secure.VOLUME_HUSH_GESTURE, VOLUME_HUSH_MUTE); - mController.onSwitchChanged(null, true); + mController.onCheckedChanged(null, true); assertThat(Settings.Secure.getInt(mContext.getContentResolver(), Settings.Secure.VOLUME_HUSH_GESTURE, UNKNOWN)).isEqualTo(VOLUME_HUSH_MUTE); @@ -151,7 +151,7 @@ public class PreventRingingSwitchPreferenceControllerTest { Settings.Secure.putInt(mContext.getContentResolver(), Settings.Secure.VOLUME_HUSH_GESTURE, VOLUME_HUSH_VIBRATE); - mController.onSwitchChanged(null, true); + mController.onCheckedChanged(null, true); assertThat(Settings.Secure.getInt(mContext.getContentResolver(), Settings.Secure.VOLUME_HUSH_GESTURE, UNKNOWN)).isEqualTo(VOLUME_HUSH_VIBRATE); diff --git a/tests/robotests/src/com/android/settings/location/LocationSwitchBarControllerTest.java b/tests/robotests/src/com/android/settings/location/LocationSwitchBarControllerTest.java index 50f618f5ffe..1f20d59dcfd 100644 --- a/tests/robotests/src/com/android/settings/location/LocationSwitchBarControllerTest.java +++ b/tests/robotests/src/com/android/settings/location/LocationSwitchBarControllerTest.java @@ -85,14 +85,14 @@ public class LocationSwitchBarControllerTest { @Test public void onSwitchChanged_switchChecked_shouldSetLocationEnabled() { - mController.onSwitchChanged(mSwitch, true); + mController.onCheckedChanged(mSwitch, true); verify(mEnabler).setLocationEnabled(true); } @Test public void onSwitchChanged_switchUnchecked_shouldSetLocationDisabled() { - mController.onSwitchChanged(mSwitch, false); + mController.onCheckedChanged(mSwitch, false); verify(mEnabler).setLocationEnabled(false); } diff --git a/tests/robotests/src/com/android/settings/notification/BubbleNotificationPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/notification/BubbleNotificationPreferenceControllerTest.java index b346b01277d..b2759767112 100644 --- a/tests/robotests/src/com/android/settings/notification/BubbleNotificationPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/notification/BubbleNotificationPreferenceControllerTest.java @@ -112,7 +112,7 @@ public class BubbleNotificationPreferenceControllerTest { public void onSwitchChanged_true_settingIsOff_flagShouldOn() { Settings.Global.putInt(mContext.getContentResolver(), NOTIFICATION_BUBBLES, OFF); - mController.onSwitchChanged(mSwitch, true); + mController.onCheckedChanged(mSwitch, true); assertThat(Settings.Global.getInt(mContext.getContentResolver(), NOTIFICATION_BUBBLES, OFF)).isEqualTo(ON); @@ -122,7 +122,7 @@ public class BubbleNotificationPreferenceControllerTest { public void onSwitchChanged_false_settingIsOn_flagShouldOff() { Settings.Global.putInt(mContext.getContentResolver(), NOTIFICATION_BUBBLES, ON); - mController.onSwitchChanged(mSwitch, false); + mController.onCheckedChanged(mSwitch, false); assertThat(Settings.Global.getInt(mContext.getContentResolver(), NOTIFICATION_BUBBLES, ON)).isEqualTo(OFF); diff --git a/tests/robotests/src/com/android/settings/notification/app/BlockPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/notification/app/BlockPreferenceControllerTest.java index 9e9e655e3af..abcf2de7504 100644 --- a/tests/robotests/src/com/android/settings/notification/app/BlockPreferenceControllerTest.java +++ b/tests/robotests/src/com/android/settings/notification/app/BlockPreferenceControllerTest.java @@ -101,7 +101,7 @@ public class BlockPreferenceControllerTest { @Test public void testNoCrashIfNoOnResume() { mController.isAvailable(); - mController.onSwitchChanged(null, false); + mController.onCheckedChanged(null, false); } @Test @@ -355,11 +355,11 @@ public class BlockPreferenceControllerTest { mController.onResume(appRow, channel, null, null, null, null, null); mController.updateState(mPreference); - mController.onSwitchChanged(null, false); + mController.onCheckedChanged(null, false); assertEquals(IMPORTANCE_NONE, channel.getImportance()); assertTrue(appRow.banned); - mController.onSwitchChanged(null, true); + mController.onCheckedChanged(null, true); assertEquals(IMPORTANCE_UNSPECIFIED, channel.getImportance()); assertFalse(appRow.banned); @@ -377,10 +377,10 @@ public class BlockPreferenceControllerTest { mController.onResume(appRow, channel, null, null, null, null, null); mController.updateState(mPreference); - mController.onSwitchChanged(null, false); + mController.onCheckedChanged(null, false); assertEquals(IMPORTANCE_NONE, channel.getImportance()); - mController.onSwitchChanged(null, true); + mController.onCheckedChanged(null, true); assertEquals(IMPORTANCE_HIGH, channel.getImportance()); verify(mBackend, times(2)).updateChannel(any(), anyInt(), any()); diff --git a/tests/robotests/src/com/android/settings/wifi/calling/WifiCallingSettingsForSubTest.java b/tests/robotests/src/com/android/settings/wifi/calling/WifiCallingSettingsForSubTest.java index 1faa611b970..2e6189ea16d 100644 --- a/tests/robotests/src/com/android/settings/wifi/calling/WifiCallingSettingsForSubTest.java +++ b/tests/robotests/src/com/android/settings/wifi/calling/WifiCallingSettingsForSubTest.java @@ -292,7 +292,7 @@ public class WifiCallingSettingsForSubTest { public void onSwitchChanged_enableSetting_shouldLaunchWfcDisclaimerFragment() { final ArgumentCaptor intentCaptor = ArgumentCaptor.forClass(Intent.class); - mFragment.onSwitchChanged(null, true); + mFragment.onCheckedChanged(null, true); // Check the WFC disclaimer fragment is launched. verify(mFragment).startActivityForResult(intentCaptor.capture(), @@ -337,7 +337,7 @@ public class WifiCallingSettingsForSubTest { @Test public void onSwitchChanged_disableSetting_shouldNotLaunchWfcDisclaimerFragment() { - mFragment.onSwitchChanged(null, false); + mFragment.onCheckedChanged(null, false); // Check the WFC disclaimer fragment is not launched. verify(mFragment, never()).startActivityForResult(any(Intent.class), anyInt()); diff --git a/tests/robotests/src/com/android/settings/wifi/tether/WifiTetherSwitchBarControllerTest.java b/tests/robotests/src/com/android/settings/wifi/tether/WifiTetherSwitchBarControllerTest.java index ca0247fdd05..0982f26d76c 100644 --- a/tests/robotests/src/com/android/settings/wifi/tether/WifiTetherSwitchBarControllerTest.java +++ b/tests/robotests/src/com/android/settings/wifi/tether/WifiTetherSwitchBarControllerTest.java @@ -150,7 +150,7 @@ public class WifiTetherSwitchBarControllerTest { public void onSwitchChanged_switchNotEnabled_doNothingForTethering() { when(mSwitch.isEnabled()).thenReturn(false); - mController.onSwitchChanged(mSwitch, mSwitch.isChecked()); + mController.onCheckedChanged(mSwitch, true); verify(mConnectivityManager, never()).startTethering(anyInt(), anyBoolean(), any(), any()); verify(mConnectivityManager, never()).stopTethering(anyInt()); @@ -158,9 +158,7 @@ public class WifiTetherSwitchBarControllerTest { @Test public void onSwitchChanged_isChecked_startTethering() { - when(mSwitch.isChecked()).thenReturn(true); - - mController.onSwitchChanged(mSwitch, mSwitch.isChecked()); + mController.onCheckedChanged(mSwitch, true); verify(mConnectivityManager).startTethering(anyInt(), anyBoolean(), any(), any()); } @@ -168,9 +166,7 @@ public class WifiTetherSwitchBarControllerTest { @Test public void onSwitchChanged_isNotChecked_stopTethering() { when(mWifiManager.getWifiApState()).thenReturn(WIFI_AP_STATE_ENABLED); - when(mSwitch.isChecked()).thenReturn(false); - - mController.onSwitchChanged(mSwitch, mSwitch.isChecked()); + mController.onCheckedChanged(mSwitch, false); verify(mConnectivityManager).stopTethering(anyInt()); }