From: Jason Monk <jmonk@google.com>
Date: Thu, 25 Jun 2015 19:03:26 +0000 (-0400)
Subject: Fix accessibility for account sync
X-Git-Tag: android-x86-6.0-r1~314
X-Git-Url: http://git.osdn.net/view?a=commitdiff_plain;h=fb6e1186;p=android-x86%2Fpackages-apps-Settings.git

Fix accessibility for account sync

Needed to get views recycling so that feedback could occur during
switch turning off.

Bug: 20020216
Change-Id: Ib7b1cdca78f5ac5064cdf521122ff7a5ee49f572
---

diff --git a/res/values/styles.xml b/res/values/styles.xml
index ef1dbc8b97..65b5d7767a 100644
--- a/res/values/styles.xml
+++ b/res/values/styles.xml
@@ -97,6 +97,10 @@
         <item name="android:layout">@layout/preference_material_settings</item>
     </style>
 
+    <style name="SyncSwitchPreference" parent="Preference">
+        <item name="android:widgetLayout">@layout/preference_widget_sync_toggle</item>
+    </style>
+
     <style name="PreferenceHeaderPanelSinglePane">
         <item name="android:layout_marginStart">0dp</item>
         <item name="android:layout_marginEnd">0dp</item>
diff --git a/res/xml/account_sync_settings.xml b/res/xml/account_sync_settings.xml
index 0ec46b739e..ab274eb087 100644
--- a/res/xml/account_sync_settings.xml
+++ b/res/xml/account_sync_settings.xml
@@ -16,4 +16,9 @@
 
 <PreferenceScreen xmlns:android="http://schemas.android.com/apk/res/android">
 
+    <!-- Needed so PreferenceGroupAdapter allows SyncStateSwitchPreference to be
+         recycled. Removed in onResume -->
+    <com.android.settings.accounts.SyncStateSwitchPreference
+        android:key="dummy" />
+
 </PreferenceScreen>
diff --git a/src/com/android/settings/accounts/AccountSyncSettings.java b/src/com/android/settings/accounts/AccountSyncSettings.java
index 2de8e4abe1..d1df82a6c8 100644
--- a/src/com/android/settings/accounts/AccountSyncSettings.java
+++ b/src/com/android/settings/accounts/AccountSyncSettings.java
@@ -149,6 +149,8 @@ public class AccountSyncSettings extends AccountPreferenceBase {
     @Override
     public void onCreate(Bundle icicle) {
         super.onCreate(icicle);
+        setPreferenceScreen(null);
+        addPreferencesFromResource(R.xml.account_sync_settings);
 
         setHasOptionsMenu(true);
     }
@@ -167,8 +169,6 @@ public class AccountSyncSettings extends AccountPreferenceBase {
     }
 
     protected void initializeUi(final View rootView) {
-        addPreferencesFromResource(R.xml.account_sync_settings);
-
         mErrorInfoView = (TextView) rootView.findViewById(R.id.sync_settings_error_info);
         mErrorInfoView.setVisibility(View.GONE);
 
@@ -202,6 +202,7 @@ public class AccountSyncSettings extends AccountPreferenceBase {
 
     @Override
     public void onResume() {
+        removePreference("dummy");
         mAuthenticatorHelper.listenToAccountUpdates();
         updateAuthDescriptions();
         onAccountsUpdate(UserHandle.getCallingUserHandle());
@@ -540,7 +541,6 @@ public class AccountSyncSettings extends AccountPreferenceBase {
             mProviderIcon.setImageDrawable(getDrawableForType(mAccount.type));
             mProviderId.setText(getLabelForType(mAccount.type));
         }
-        addPreferencesFromResource(R.xml.account_sync_settings);
     }
 
     @Override
diff --git a/src/com/android/settings/accounts/SyncStateSwitchPreference.java b/src/com/android/settings/accounts/SyncStateSwitchPreference.java
index 4d6387096d..fd0c45c810 100644
--- a/src/com/android/settings/accounts/SyncStateSwitchPreference.java
+++ b/src/com/android/settings/accounts/SyncStateSwitchPreference.java
@@ -23,6 +23,7 @@ import android.preference.SwitchPreference;
 import android.util.AttributeSet;
 import android.util.Log;
 import android.view.View;
+import android.view.ViewGroup;
 import android.widget.TextView;
 
 import com.android.settings.R;
@@ -43,17 +44,15 @@ public class SyncStateSwitchPreference extends SwitchPreference {
     private boolean mOneTimeSyncMode = false;
 
     public SyncStateSwitchPreference(Context context, AttributeSet attrs) {
-        super(context, attrs);
-        setWidgetLayoutResource(R.layout.preference_widget_sync_toggle);
+        super(context, attrs, 0, R.style.SyncSwitchPreference);
         mAccount = null;
         mAuthority = null;
     }
 
     public SyncStateSwitchPreference(Context context, Account account, String authority) {
-        super(context, null);
+        super(context, null, 0, R.style.SyncSwitchPreference);
         mAccount = account;
         mAuthority = authority;
-        setWidgetLayoutResource(R.layout.preference_widget_sync_toggle);
     }
 
     @Override