commit 70f291c40c66463d6ea8b76b28d0b6900682ccab
author Wenyi Wang <wenyiw@google.com> Fri Feb 17 10:34:09 2017 -0800
committer Wenyi Wang <wenyiw@google.com> Fri Feb 17 10:34:32 2017 -0800
tree 74aa21b77de58f0d3b4b271d25224ea30753a366
parent ffcb31f6ece6257d0bc50aa572de0d4b93c82e97

Build list items first and then notify data set changed

Bug: 35313285

Test: ran the following command 10 times and verified the
      crash didn't happen. Before the CL, usually there are 2-3 times
      of crash out of 10 attempts.

adb shell monkey -p
com.google.android.contacts --ignore-security-exceptions
--monitor-native-crashes -s 493 -v -v -v 125000

Change-Id: Ic2ab30c55e9c6eb729f163b0ed11174f6f6340d4

src/com/android/contacts/drawer/DrawerAdapter.java

diff --git a/src/com/android/contacts/drawer/DrawerAdapter.java b/src/com/android/contacts/drawer/DrawerAdapter.java
index 54cd669..fcb41c6 100644
--- a/src/com/android/contacts/drawer/DrawerAdapter.java
+++ b/src/com/android/contacts/drawer/DrawerAdapter.java
@@ -336,8 +336,8 @@
     }
 
     private void notifyChangeAndRebuildList() {
-        notifyDataSetChanged();
         rebuildItemsList();
+        notifyDataSetChanged();
     }
 
     public void setSelectedContactsView(ContactsView contactsView) {