diff --git a/src/com/android/settings/display/NightDisplayActivationPreferenceController.java b/src/com/android/settings/display/NightDisplayActivationPreferenceController.java index b08458ad532..4df3d085616 100644 --- a/src/com/android/settings/display/NightDisplayActivationPreferenceController.java +++ b/src/com/android/settings/display/NightDisplayActivationPreferenceController.java @@ -19,27 +19,20 @@ package com.android.settings.display; import android.content.Context; import android.hardware.display.ColorDisplayManager; import android.text.TextUtils; -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.overlay.FeatureFactory; import com.android.settingslib.core.instrumentation.MetricsFeatureProvider; -import com.android.settingslib.widget.MainSwitchPreference; /** * Controller that updates the night display. */ -public class NightDisplayActivationPreferenceController extends - TogglePreferenceController implements OnCheckedChangeListener { +public class NightDisplayActivationPreferenceController extends TogglePreferenceController { private final MetricsFeatureProvider mMetricsFeatureProvider; private ColorDisplayManager mColorDisplayManager; private NightDisplayTimeFormatter mTimeFormatter; - private MainSwitchPreference mPreference; public NightDisplayActivationPreferenceController(Context context, String key) { super(context, key); @@ -70,25 +63,6 @@ public class NightDisplayActivationPreferenceController extends return R.string.menu_key_display; } - @Override - public void displayPreference(PreferenceScreen screen) { - super.displayPreference(screen); - - mPreference = (MainSwitchPreference) screen.findPreference(getPreferenceKey()); - mPreference.addOnSwitchChangeListener(this); - mPreference.setChecked(mColorDisplayManager.isNightDisplayActivated()); - } - - @Override - 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 - // control the toggle preference. - setChecked(isChecked); - } - } - /** FOR SLICES */ @Override diff --git a/tests/robotests/src/com/android/settings/display/NightDisplayActivationPreferenceControllerTest.java b/tests/robotests/src/com/android/settings/display/NightDisplayActivationPreferenceControllerTest.java index 5c38411855c..fed0cd6bde8 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.onCheckedChanged(null, true); + controller.setChecked(true); assertThat(mColorDisplayManager.isNightDisplayActivated()).isEqualTo(true); } @@ -115,7 +115,7 @@ public class NightDisplayActivationPreferenceControllerTest { final NightDisplayActivationPreferenceController controller = new NightDisplayActivationPreferenceController(mContext, "night_display_activated"); - controller.onCheckedChanged(null, false); + controller.setChecked(false); assertThat(mColorDisplayManager.isNightDisplayActivated()).isEqualTo(false); }