commit 79f9fa073cf3311b552a0a055a44a26c06e395d3
author Hung-ying Tyan <tyanh@google.com> Sat Aug 01 10:08:03 2009 +0800
committer Hung-ying Tyan <tyanh@google.com> Sat Aug 01 10:08:03 2009 +0800
tree dc86f5f6dd2aaac2fd1a0e29d76f5baf363bb407
parent 7be218e7f7bb57049d9e3586cbd65cfc8f218f9d

Reduce the number of keystore connections.

src/com/android/settings/SecuritySettings.java

diff --git a/src/com/android/settings/SecuritySettings.java b/src/com/android/settings/SecuritySettings.java
index 5a5804e..7f8cec7 100644
--- a/src/com/android/settings/SecuritySettings.java
+++ b/src/com/android/settings/SecuritySettings.java
@@ -233,9 +233,10 @@
         PreferenceCategory credStoreCat = new PreferenceCategory(this);
         credStoreCat.setTitle(R.string.cstor_settings_category);
         root.addPreference(credStoreCat);
-        credStoreCat.addPreference(mCstorHelper.createAccessCheckBox());
+        boolean i = mCstorHelper.isCstorInitialized();
+        credStoreCat.addPreference(mCstorHelper.createAccessCheckBox(i));
         credStoreCat.addPreference(mCstorHelper.createSetPasswordPreference());
-        credStoreCat.addPreference(mCstorHelper.createResetPreference());
+        credStoreCat.addPreference(mCstorHelper.createResetPreference(i));
 
         return root;
     }
@@ -793,12 +794,12 @@
             mResetButton.setEnabled(enabled);
         }
 
-        private Preference createAccessCheckBox() {
+        private Preference createAccessCheckBox(boolean isInitialized) {
             CheckBoxPreference pref = new CheckBoxPreference(
                     SecuritySettings.this);
             pref.setTitle(R.string.cstor_access_title);
             pref.setSummary(R.string.cstor_access_summary);
-            pref.setChecked(isCstorUnlocked());
+            if (isInitialized) pref.setChecked(isCstorUnlocked());
             pref.setOnPreferenceChangeListener(
                     new Preference.OnPreferenceChangeListener() {
                         public boolean onPreferenceChange(
@@ -813,7 +814,7 @@
                             return true;
                         }
                     });
-            pref.setEnabled(isCstorInitialized());
+            pref.setEnabled(isInitialized);
             mAccessCheckBox = pref;
             return pref;
         }
@@ -834,7 +835,7 @@
             return pref;
         }
 
-        private Preference createResetPreference() {
+        private Preference createResetPreference(boolean isInitialized) {
             Preference pref = new Preference(SecuritySettings.this);
             pref.setTitle(R.string.cstor_reset_title);
             pref.setSummary(R.string.cstor_reset_summary);
@@ -845,7 +846,7 @@
                             return true;
                         }
                     });
-            pref.setEnabled(isCstorInitialized());
+            pref.setEnabled(isInitialized);
             mResetButton = pref;
             return pref;
         }