diff options
author | Brad Fitzpatrick <bradfitz@android.com> | 2010-08-30 18:18:54 -0700 |
---|---|---|
committer | Android Git Automerger <android-git-automerger@android.com> | 2010-08-30 18:18:54 -0700 |
commit | 734d270769c609c7d3a1f2ec46e1d329d9682ab4 (patch) | |
tree | a6c455a9a3b31eb0ecdbc591b2750f4d29ff5b20 /core/java | |
parent | 03e83d4ad909f5c07fb2011e03348a413453e909 (diff) | |
parent | bb660d7e1bed3a07a1804bd9641d9634bfaa4972 (diff) | |
download | frameworks_base-734d270769c609c7d3a1f2ec46e1d329d9682ab4.zip frameworks_base-734d270769c609c7d3a1f2ec46e1d329d9682ab4.tar.gz frameworks_base-734d270769c609c7d3a1f2ec46e1d329d9682ab4.tar.bz2 |
am bb660d7e: Merge "Renaming SharedPreferences.startCommit to apply" into gingerbread
Merge commit 'bb660d7e1bed3a07a1804bd9641d9634bfaa4972' into gingerbread-plus-aosp
* commit 'bb660d7e1bed3a07a1804bd9641d9634bfaa4972':
Renaming SharedPreferences$Editor.startCommit to apply
Diffstat (limited to 'core/java')
-rw-r--r-- | core/java/android/app/ContextImpl.java | 14 | ||||
-rw-r--r-- | core/java/android/content/SharedPreferences.java | 24 | ||||
-rw-r--r-- | core/java/android/preference/Preference.java | 2 | ||||
-rw-r--r-- | core/java/android/server/BluetoothService.java | 2 |
4 files changed, 17 insertions, 25 deletions
diff --git a/core/java/android/app/ContextImpl.java b/core/java/android/app/ContextImpl.java index f33dcf5..28ae3ca 100644 --- a/core/java/android/app/ContextImpl.java +++ b/core/java/android/app/ContextImpl.java @@ -121,7 +121,6 @@ import java.util.Set; import java.util.WeakHashMap; import java.util.concurrent.CountDownLatch; import java.util.concurrent.ExecutorService; -import java.util.concurrent.atomic.AtomicBoolean; class ReceiverRestrictedContext extends ContextWrapper { ReceiverRestrictedContext(Context base) { @@ -2818,8 +2817,6 @@ class ContextImpl extends Context { private final Map<String, Object> mModified = Maps.newHashMap(); private boolean mClear = false; - private AtomicBoolean mCommitInFlight = new AtomicBoolean(false); - public Editor putString(String key, String value) { synchronized (this) { mModified.put(key, value); @@ -2865,11 +2862,7 @@ class ContextImpl extends Context { } } - public void startCommit() { - if (!mCommitInFlight.compareAndSet(false, true)) { - throw new IllegalStateException("can't call startCommit() twice"); - } - + public void apply() { final MemoryCommitResult mcr = commitToMemory(); final Runnable awaitCommit = new Runnable() { public void run() { @@ -2885,7 +2878,6 @@ class ContextImpl extends Context { Runnable postWriteRunnable = new Runnable() { public void run() { awaitCommit.run(); - mCommitInFlight.set(false); QueuedWork.remove(awaitCommit); } }; @@ -3009,13 +3001,13 @@ class ContextImpl extends Context { * that they're enqueued. * * @param postWriteRunnable if non-null, we're being called - * from startCommit() and this is the runnable to run after + * from apply() and this is the runnable to run after * the write proceeds. if null (from a regular commit()), * then we're allowed to do this disk write on the main * thread (which in addition to reducing allocations and * creating a background thread, this has the advantage that * we catch them in userdebug StrictMode reports to convert - * them where possible to startCommit...) + * them where possible to apply() ...) */ private void enqueueDiskWrite(final MemoryCommitResult mcr, final Runnable postWriteRunnable) { diff --git a/core/java/android/content/SharedPreferences.java b/core/java/android/content/SharedPreferences.java index b3db2ac..1484204 100644 --- a/core/java/android/content/SharedPreferences.java +++ b/core/java/android/content/SharedPreferences.java @@ -54,13 +54,13 @@ public interface SharedPreferences { /** * Interface used for modifying values in a {@link SharedPreferences} * object. All changes you make in an editor are batched, and not copied - * back to the original {@link SharedPreferences} or persistent storage - * until you call {@link #commit}. + * back to the original {@link SharedPreferences} until you call {@link #commit} + * or {@link #apply} */ public interface Editor { /** * Set a String value in the preferences editor, to be written back once - * {@link #commit} is called. + * {@link #commit} or {@link #apply} are called. * * @param key The name of the preference to modify. * @param value The new value for the preference. @@ -72,7 +72,7 @@ public interface SharedPreferences { /** * Set an int value in the preferences editor, to be written back once - * {@link #commit} is called. + * {@link #commit} or {@link #apply} are called. * * @param key The name of the preference to modify. * @param value The new value for the preference. @@ -84,7 +84,7 @@ public interface SharedPreferences { /** * Set a long value in the preferences editor, to be written back once - * {@link #commit} is called. + * {@link #commit} or {@link #apply} are called. * * @param key The name of the preference to modify. * @param value The new value for the preference. @@ -96,7 +96,7 @@ public interface SharedPreferences { /** * Set a float value in the preferences editor, to be written back once - * {@link #commit} is called. + * {@link #commit} or {@link #apply} are called. * * @param key The name of the preference to modify. * @param value The new value for the preference. @@ -108,7 +108,7 @@ public interface SharedPreferences { /** * Set a boolean value in the preferences editor, to be written back - * once {@link #commit} is called. + * once {@link #commit} or {@link #apply} are called. * * @param key The name of the preference to modify. * @param value The new value for the preference. @@ -159,7 +159,7 @@ public interface SharedPreferences { * * <p>If you don't care about the return value and you're * using this from your application's main thread, consider - * using {@link #startCommit} instead. + * using {@link #apply} instead. * * @return Returns true if the new values were successfully written * to persistent storage. @@ -173,16 +173,16 @@ public interface SharedPreferences { * in the SharedPreferences. * * <p>Note that when two editors are modifying preferences at the same - * time, the last one to call commit wins. + * time, the last one to call apply wins. * * <p>Unlike {@link #commit}, which writes its preferences out - * to persistent storage synchronously, {@link #startCommit} + * to persistent storage synchronously, {@link #apply} * commits its changes to the in-memory * {@link SharedPreferences} immediately but starts an * asynchronous commit to disk and you won't be notified of * any failures. If another editor on this * {@link SharedPreferences} does a regular {@link #commit} - * while a {@link #startCommit} is still outstanding, the + * while a {@link #apply} is still outstanding, the * {@link #commit} will block until all async commits are * completed as well as the commit itself. * @@ -190,7 +190,7 @@ public interface SharedPreferences { * the base class will wait for any async commits to finish in * its {@link android.app.Activity#onPause}.</p> */ - void startCommit(); + void apply(); } /** diff --git a/core/java/android/preference/Preference.java b/core/java/android/preference/Preference.java index 1453329..dde6493 100644 --- a/core/java/android/preference/Preference.java +++ b/core/java/android/preference/Preference.java @@ -1195,7 +1195,7 @@ public class Preference implements Comparable<Preference>, OnDependencyChangeLis private void tryCommit(SharedPreferences.Editor editor) { if (mPreferenceManager.shouldCommit()) { - editor.startCommit(); + editor.apply(); } } diff --git a/core/java/android/server/BluetoothService.java b/core/java/android/server/BluetoothService.java index abd66ae..b2aa04d 100644 --- a/core/java/android/server/BluetoothService.java +++ b/core/java/android/server/BluetoothService.java @@ -1737,7 +1737,7 @@ public class BluetoothService extends IBluetooth.Stub { mContext.getSharedPreferences(SHARED_PREFERENCES_NAME, mContext.MODE_PRIVATE).edit(); editor.putBoolean(SHARED_PREFERENCE_DOCK_ADDRESS + mDockAddress, true); - editor.startCommit(); + editor.apply(); } } } |