diff options
| author | Tony Mantler <nicoya@google.com> | 2014-10-16 15:36:50 -0700 |
|---|---|---|
| committer | Tony Mantler <nicoya@google.com> | 2014-10-16 15:36:50 -0700 |
| commit | 46b5b33c4de296b5c3b61b031db9d36bf429be6a (patch) | |
| tree | 8725340fcdea76ee742a56028c297f3b90dc8c7e /src | |
| parent | 7a2a78faea3a0e7d92f4da94f95d9c59fc654af6 (diff) | |
| download | android_packages_apps_UnifiedEmail-46b5b33c4de296b5c3b61b031db9d36bf429be6a.tar.gz android_packages_apps_UnifiedEmail-46b5b33c4de296b5c3b61b031db9d36bf429be6a.tar.bz2 android_packages_apps_UnifiedEmail-46b5b33c4de296b5c3b61b031db9d36bf429be6a.zip | |
Prevent rare NPE
b/18017065
Change-Id: I3f6c8bfcd0650adc32a601120f3dfade23eff0cc
Diffstat (limited to 'src')
| -rw-r--r-- | src/com/android/mail/ui/AbstractActivityController.java | 4 | ||||
| -rw-r--r-- | src/com/android/mail/ui/RecentFolderList.java | 4 |
2 files changed, 5 insertions, 3 deletions
diff --git a/src/com/android/mail/ui/AbstractActivityController.java b/src/com/android/mail/ui/AbstractActivityController.java index ff97f586e..a7de33b60 100644 --- a/src/com/android/mail/ui/AbstractActivityController.java +++ b/src/com/android/mail/ui/AbstractActivityController.java @@ -940,7 +940,9 @@ public abstract class AbstractActivityController implements ActivityController, setListContext(folder, query); showConversationList(mConvListContext); // Touch the current folder: it is different, and it has been accessed. - mRecentFolderList.touchFolder(mFolder, mAccount); + if (mFolder != null) { + mRecentFolderList.touchFolder(mFolder, mAccount); + } } resetActionBarIcon(); } diff --git a/src/com/android/mail/ui/RecentFolderList.java b/src/com/android/mail/ui/RecentFolderList.java index b581aafbe..f2de5892b 100644 --- a/src/com/android/mail/ui/RecentFolderList.java +++ b/src/com/android/mail/ui/RecentFolderList.java @@ -20,6 +20,7 @@ import android.content.ContentValues; import android.content.Context; import android.net.Uri; import android.os.AsyncTask; +import android.support.annotation.NonNull; import com.android.mail.content.ObjectCursor; import com.android.mail.providers.Account; @@ -189,7 +190,7 @@ public final class RecentFolderList { * be called with a null folder. * @param folder the folder we touched */ - public void touchFolder(Folder folder, Account account) { + public void touchFolder(@NonNull Folder folder, Account account) { // We haven't got a valid account yet, cannot proceed. if (mAccount == null || !mAccount.equals(account)) { if (account != null) { @@ -199,7 +200,6 @@ public final class RecentFolderList { return; } } - assert (folder != null); if (folder.isProviderFolder() || folder.isType(FolderType.SEARCH)) { LogUtils.d(TAG, "Not touching recent folder because it's provider or search folder"); |