Merge change 26821 into eclair

* changes:
  Make removeOnAccountsUpdateListener cope with null handlers.
This commit is contained in:
Android (Google) Code Review
2009-09-24 17:29:53 -04:00

View File

@@ -859,9 +859,10 @@ public class AccountManager {
throw new IllegalArgumentException("the listener is null"); throw new IllegalArgumentException("the listener is null");
} }
synchronized (mAccountsUpdatedListeners) { synchronized (mAccountsUpdatedListeners) {
if (mAccountsUpdatedListeners.remove(listener) == null) { if (!mAccountsUpdatedListeners.containsKey(listener)) {
throw new IllegalStateException("this listener was not previously added"); throw new IllegalStateException("this listener was not previously added");
} }
mAccountsUpdatedListeners.remove(listener);
if (mAccountsUpdatedListeners.isEmpty()) { if (mAccountsUpdatedListeners.isEmpty()) {
mContext.unregisterReceiver(mAccountsChangedBroadcastReceiver); mContext.unregisterReceiver(mAccountsChangedBroadcastReceiver);
} }