From b4558c960793d386b023c23f99725c0500c18b4b Mon Sep 17 00:00:00 2001
From: menghanli <menghanli@google.com>
Date: Wed, 21 Apr 2021 10:12:56 +0800
Subject: [PATCH] [Material Next] Update main feature switch for Caption
 preferences.

Fix: 185950803
Test: manual test
Change-Id: I5431461f3088652ddff7400b24b3eea25ddf5f9a
---
 res/xml/captioning_settings.xml               | 14 ++++++--------
 .../CaptionPropertiesFragment.java            | 19 +++++++++++++++----
 2 files changed, 21 insertions(+), 12 deletions(-)

diff --git a/res/xml/captioning_settings.xml b/res/xml/captioning_settings.xml
index 6bd3763e191..a58e2243075 100644
--- a/res/xml/captioning_settings.xml
+++ b/res/xml/captioning_settings.xml
@@ -21,22 +21,20 @@
     android:persistent="false"
     android:title="@string/accessibility_captioning_title">
 
+    <com.android.settings.widget.SettingsMainSwitchPreference
+        android:key="captioning_preference_switch"
+        android:persistent="false"
+        android:summary="@string/accessibility_caption_primary_switch_summary"
+        android:title="@string/accessibility_caption_primary_switch_title" />
+
     <com.android.settingslib.widget.LayoutPreference
         android:key="captions_preview"
         android:layout="@layout/accessibility_captions_preview"
         android:persistent="false"
         android:selectable="false"
         android:title="@string/summary_placeholder"
-        settings:allowDividerBelow="true"
         settings:searchable="false" />
 
-    <SwitchPreference
-        android:key="captioning_preference_switch"
-        android:persistent="false"
-        android:summary="@string/accessibility_caption_primary_switch_summary"
-        android:title="@string/accessibility_caption_primary_switch_title"
-        settings:allowDividerAbove="true" />
-
     <Preference
         android:fragment="com.android.settings.accessibility.CaptionAppearanceFragment"
         android:key="captioning_caption_appearance"
diff --git a/src/com/android/settings/accessibility/CaptionPropertiesFragment.java b/src/com/android/settings/accessibility/CaptionPropertiesFragment.java
index 786585ad72e..01458283907 100644
--- a/src/com/android/settings/accessibility/CaptionPropertiesFragment.java
+++ b/src/com/android/settings/accessibility/CaptionPropertiesFragment.java
@@ -22,15 +22,17 @@ import android.content.Context;
 import android.os.Bundle;
 import android.provider.Settings;
 import android.view.accessibility.CaptioningManager;
+import android.widget.Switch;
 
 import androidx.preference.Preference;
 import androidx.preference.Preference.OnPreferenceChangeListener;
-import androidx.preference.SwitchPreference;
 
 import com.android.settings.R;
 import com.android.settings.SettingsPreferenceFragment;
 import com.android.settings.search.BaseSearchIndexProvider;
+import com.android.settings.widget.SettingsMainSwitchPreference;
 import com.android.settingslib.search.SearchIndexable;
+import com.android.settingslib.widget.OnMainSwitchChangeListener;
 
 import com.google.common.primitives.Floats;
 
@@ -40,14 +42,14 @@ import java.util.List;
 /** Settings fragment containing captioning properties. */
 @SearchIndexable(forTarget = SearchIndexable.ALL & ~SearchIndexable.ARC)
 public class CaptionPropertiesFragment extends SettingsPreferenceFragment
-        implements OnPreferenceChangeListener {
+        implements OnPreferenceChangeListener, OnMainSwitchChangeListener {
     private static final String PREF_SWITCH = "captioning_preference_switch";
     private static final String PREF_TEXT = "captioning_caption_appearance";
     private static final String PREF_MORE = "captioning_more_options";
 
     private CaptioningManager mCaptioningManager;
 
-    private SwitchPreference mSwitch;
+    private SettingsMainSwitchPreference mSwitch;
     private Preference mTextAppearance;
     private Preference mMoreOptions;
 
@@ -78,7 +80,7 @@ public class CaptionPropertiesFragment extends SettingsPreferenceFragment
     }
 
     private void initializeAllPreferences() {
-        mSwitch = (SwitchPreference) findPreference(PREF_SWITCH);
+        mSwitch = (SettingsMainSwitchPreference) findPreference(PREF_SWITCH);
         mTextAppearance = (Preference) findPreference(PREF_TEXT);
         mMoreOptions = (Preference) findPreference(PREF_MORE);
 
@@ -88,6 +90,8 @@ public class CaptionPropertiesFragment extends SettingsPreferenceFragment
 
     private void installUpdateListeners() {
         mSwitch.setOnPreferenceChangeListener(this);
+        mSwitch.addOnSwitchChangeListener(this);
+
     }
 
     private void initFontSizeValuesArray() {
@@ -133,4 +137,11 @@ public class CaptionPropertiesFragment extends SettingsPreferenceFragment
 
     public static final BaseSearchIndexProvider SEARCH_INDEX_DATA_PROVIDER =
             new BaseSearchIndexProvider(R.xml.captioning_settings);
+
+    @Override
+    public void onSwitchChanged(Switch switchView, boolean isChecked) {
+        final ContentResolver cr = getActivity().getContentResolver();
+        Settings.Secure.putInt(
+                cr, Settings.Secure.ACCESSIBILITY_CAPTIONING_ENABLED, isChecked ? 1 : 0);
+    }
 }