commit c47f80f1ae96e3c8b6a750d68cc12dfbbca97254
author Adam Powell <adamp@google.com> Fri Aug 10 14:39:28 2012 -0700
committer Android (Google) Code Review <android-gerrit@google.com> Fri Aug 10 14:39:28 2012 -0700
tree 29c06be48fe4da125aa8aaffdcf241e03e9dbd60
parent f5fd82acc3a7e87c8901e7ab3d5c4f28648c149f
parent c1b07211b8a2a1eebefa8e5403ca7b2c6ccdccc6

Merge "Don't assume a default of 'false' for TwoStatePreference" into jb-mr1-dev

core/java/android/preference/TwoStatePreference.java

diff --git a/core/java/android/preference/TwoStatePreference.java b/core/java/android/preference/TwoStatePreference.java
index d186b20..c649879 100644
--- a/core/java/android/preference/TwoStatePreference.java
+++ b/core/java/android/preference/TwoStatePreference.java
@@ -37,6 +37,7 @@
     private CharSequence mSummaryOn;
     private CharSequence mSummaryOff;
     boolean mChecked;
+    private boolean mCheckedSet;
     private boolean mSendClickAccessibilityEvent;
     private boolean mDisableDependentsState;
 
@@ -74,11 +75,16 @@
      * @param checked The checked state.
      */
     public void setChecked(boolean checked) {
-        if (mChecked != checked) {
+        // Always persist/notify the first time; don't assume the field's default of false.
+        final boolean changed = mChecked != checked;
+        if (changed || !mCheckedSet) {
             mChecked = checked;
+            mCheckedSet = true;
             persistBoolean(checked);
-            notifyDependencyChange(shouldDisableDependents());
-            notifyChanged();
+            if (changed) {
+                notifyDependencyChange(shouldDisableDependents());
+                notifyChanged();
+            }
         }
     }