From d14c53989820f2d12e58bcdd6872609baed827d5 Mon Sep 17 00:00:00 2001 From: "Android (Google) Code Review" Date: Thu, 24 Sep 2009 17:29:53 -0400 Subject: [PATCH] Merge change 26821 into eclair * changes: Make removeOnAccountsUpdateListener cope with null handlers. --- core/java/android/accounts/AccountManager.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/core/java/android/accounts/AccountManager.java b/core/java/android/accounts/AccountManager.java index 1ee7f60fb9ce8..2979072ebd98b 100644 --- a/core/java/android/accounts/AccountManager.java +++ b/core/java/android/accounts/AccountManager.java @@ -859,9 +859,10 @@ public class AccountManager { throw new IllegalArgumentException("the listener is null"); } synchronized (mAccountsUpdatedListeners) { - if (mAccountsUpdatedListeners.remove(listener) == null) { + if (!mAccountsUpdatedListeners.containsKey(listener)) { throw new IllegalStateException("this listener was not previously added"); } + mAccountsUpdatedListeners.remove(listener); if (mAccountsUpdatedListeners.isEmpty()) { mContext.unregisterReceiver(mAccountsChangedBroadcastReceiver); }