samsung: Migrate to CompoundButton.OnCheckedChangeListener
authorChaohui Wang <chaohuiw@google.com>
Fri, 8 Mar 2024 09:54:40 +0000 (15:24 +0530)
committerLuK1337 <priv.luk@gmail.com>
Wed, 20 Mar 2024 15:54:28 +0000 (16:54 +0100)
Switch and SwitchCompat are both CompoundButton.

Using CompoundButton in Java will helps migration in the future.

Bug: 306658427 | AOSP
Test: manual - check Settings pages
Test: m RunSettingsLibRoboTests
Signed-off-by: RisenID <risen@pixelexperience.org>
Change-Id: I85a70d4c504d8584030ea4a058f30d74206ab835

dap/src/org/lineageos/dap/DolbyFragment.kt
doze/src/org/lineageos/settings/doze/SamsungDozeSettings.java

index 2331b65fa8b0077b1cb9dc6791b71ab59f632bae..29834343dd143d763cbfba8935308e3003d69de1 100644 (file)
 package org.lineageos.dap
 
 import android.os.Bundle
-import android.widget.Switch
+import android.widget.CompoundButton
+import android.widget.CompoundButton.OnCheckedChangeListener
 
 import androidx.preference.PreferenceFragment
 
 import com.android.settingslib.widget.MainSwitchPreference
-import com.android.settingslib.widget.OnMainSwitchChangeListener
 import com.android.settingslib.widget.RadioButtonPreference
 
 import org.lineageos.dap.R
 
-class DolbyFragment : PreferenceFragment(), OnMainSwitchChangeListener {
+class DolbyFragment : PreferenceFragment(), OnCheckedChangeListener {
 
     private lateinit var switchBar: MainSwitchPreference
 
@@ -47,7 +47,7 @@ class DolbyFragment : PreferenceFragment(), OnMainSwitchChangeListener {
         }
     }
 
-    override fun onSwitchChanged(switchView: Switch, isChecked: Boolean) {
+    override fun onCheckedChanged(buttonView: CompoundButton, isChecked: Boolean) {
         DolbyCore.setEnabled(isChecked)
     }
 
index f0f9b52525390bf3a1f3e1da4bf47e85d8ccdc45..0bc7fb84349f08a65e152f0d3b9051e66da437d7 100644 (file)
@@ -19,7 +19,8 @@ package org.lineageos.settings.doze;
 
 import android.os.Bundle;
 import android.os.Handler;
-import android.widget.Switch;
+import android.widget.CompoundButton;
+import android.widget.CompoundButton.OnCheckedChangeListener;
 
 import androidx.preference.Preference;
 import androidx.preference.Preference.OnPreferenceChangeListener;
@@ -27,12 +28,11 @@ import androidx.preference.PreferenceFragment;
 import androidx.preference.SwitchPreference;
 
 import com.android.settingslib.widget.MainSwitchPreference;
-import com.android.settingslib.widget.OnMainSwitchChangeListener;
 
 import org.lineageos.internal.util.ScreenType;
 
 public class SamsungDozeSettings extends PreferenceFragment
-        implements OnPreferenceChangeListener, OnMainSwitchChangeListener {
+        implements OnPreferenceChangeListener, OnCheckedChangeListener {
 
     private MainSwitchPreference mSwitchBar;
 
@@ -108,7 +108,7 @@ public class SamsungDozeSettings extends PreferenceFragment
     }
 
     @Override
-    public void onSwitchChanged(Switch switchView, boolean isChecked) {
+    public void onCheckedChanged(CompoundButton buttonView, boolean isChecked) {
         Utils.enableDoze(getActivity(), isChecked);
         Utils.checkDozeService(getActivity());