Introduce an ErrorCatchingListView to fix a pre-ICS ListView bug (see http://stackoverflow.com/questions/12473625/nullpointerexception-at-android-widget-abslistview-contentfitsabslistview-java)

astrid/plugin-src/com/todoroo/astrid/taskrabbit/TaskRabbitActivity.java

@@ -69,6 +69,7 @@ import com.todoroo.astrid.service.StatisticsService;
 import com.todoroo.astrid.service.TaskService;
 import com.todoroo.astrid.service.ThemeService;
 import com.todoroo.astrid.taskrabbit.TaskRabbitLocationManager.LocationResult;
+import com.todoroo.astrid.ui.ErrorCatchingListView;
 import com.todoroo.astrid.ui.FragmentPopover;
 import com.todoroo.astrid.welcome.HelpInfoPopover;
 
@@ -897,7 +898,7 @@ public class TaskRabbitActivity extends FragmentActivity {
         String[] keys = getResources().getStringArray(R.array.tr_preset_types);
         if (adapter == null) {
             adapter = new ArrayAdapter<String>(this, R.layout.task_rabbit_menu_row, keys);
-            menuList = new ListView(this);
+            menuList = new ErrorCatchingListView(this);
             menuList.setAdapter(adapter);
             menuList.setCacheColorHint(Color.TRANSPARENT);
 

astrid/res/layout-land/filter_list_activity.xml

@@ -12,7 +12,7 @@
 
 
     <!-- List -->
-    <ListView android:id="@android:id/list"
+    <com.todoroo.astrid.ui.ErrorCatchingListView android:id="@android:id/list"
         android:layout_width="fill_parent"
         android:layout_height="180dip"
         android:layout_alignParentTop="true"

astrid/res/layout-land/filter_list_fragment_alternative.xml

@@ -12,7 +12,7 @@
 
 
     <!-- List -->
-    <ListView android:id="@android:id/list"
+    <com.todoroo.astrid.ui.ErrorCatchingListView android:id="@android:id/list"
         android:layout_width="fill_parent"
         android:layout_height="230dip"
         android:layout_alignParentTop="true"

astrid/res/layout/addon_list_container.xml

@@ -18,7 +18,7 @@
         android:text="@string/AOA_no_addons"
         style="@style/TextAppearance.TLA_NoItems" />
   
-    <ListView android:id="@+id/list"
+    <com.todoroo.astrid.ui.ErrorCatchingListView android:id="@+id/list"
         android:paddingRight="8dip"
         android:orientation="vertical"
         android:layout_width="fill_parent"

astrid/res/layout/control_set_assigned.xml

@@ -45,7 +45,7 @@
           android:gravity="center_vertical" />
     </LinearLayout>
     
-	<ListView
+	<com.todoroo.astrid.ui.ErrorCatchingListView
 	    android:id="@+id/assigned_list"
 	    android:layout_width="fill_parent"
 	    android:layout_height="fill_parent"

astrid/res/layout/control_set_tags.xml

@@ -25,7 +25,7 @@
       android:layout_height="1px"
       android:padding="5dip"
       android:background="@android:drawable/divider_horizontal_dark"/>  
-  <ListView
+  <com.todoroo.astrid.ui.ErrorCatchingListView
       android:id="@+id/existingTags"
       android:layout_width="fill_parent"
       android:layout_height="fill_parent"

astrid/res/layout/custom_filter_activity.xml

@@ -20,7 +20,7 @@
         android:capitalize="sentences" />
     
     <!-- List -->
-    <ListView android:id="@android:id/list"
+    <com.todoroo.astrid.ui.ErrorCatchingListView android:id="@android:id/list"
         android:layout_width="fill_parent"
         android:layout_height="fill_parent"
         android:layout_weight="100"

astrid/res/layout/filter_list_activity.xml

@@ -12,7 +12,7 @@
 
 
     <!-- List -->
-    <ListView android:id="@android:id/list"
+    <com.todoroo.astrid.ui.ErrorCatchingListView android:id="@android:id/list"
         android:layout_width="fill_parent"
         android:layout_height="320dip"
         android:layout_alignParentTop="true"

astrid/res/layout/filter_list_activity_3pane.xml

@@ -35,7 +35,7 @@
     </LinearLayout>
 
     <!-- List -->
-    <ListView android:id="@android:id/list"
+    <com.todoroo.astrid.ui.ErrorCatchingListView android:id="@android:id/list"
         android:layout_width="fill_parent"
         android:layout_height="fill_parent"
         android:layout_weight="100"

astrid/res/layout/filter_list_fragment_alternative.xml

@@ -12,7 +12,7 @@
 
 
     <!-- List -->
-    <ListView android:id="@android:id/list"
+    <com.todoroo.astrid.ui.ErrorCatchingListView android:id="@android:id/list"
         android:layout_width="fill_parent"
         android:layout_height="370dip"
         android:layout_alignParentTop="true"

astrid/res/layout/filter_list_fragment_alternative_3pane.xml

@@ -12,7 +12,7 @@
 
 
     <!-- List -->
-    <ListView android:id="@android:id/list"
+    <com.todoroo.astrid.ui.ErrorCatchingListView android:id="@android:id/list"
         android:layout_width="fill_parent"
         android:layout_height="fill_parent"
         android:layout_alignParentTop="true"

astrid/res/layout/gtasks_login_activity.xml

@@ -9,7 +9,7 @@
     android:layout_height="fill_parent"
     style="@style/Content">
 
-  <ListView android:id="@android:id/list"
+  <com.todoroo.astrid.ui.ErrorCatchingListView android:id="@android:id/list"
         android:layout_width="fill_parent"
         android:layout_height="fill_parent"/>
         

astrid/res/layout/locale_edit_alerts.xml

@@ -34,7 +34,7 @@
                 android:layout_margin="10px"
                 android:background="@drawable/locale_border">
     
-                <ListView android:id="@android:id/list"
+                <com.todoroo.astrid.ui.ErrorCatchingListView android:id="@android:id/list"
                     android:layout_width="fill_parent"
                     android:layout_height="fill_parent"
                     android:scrollbars="vertical"

astrid/res/layout/tag_updates_fragment.xml

@@ -38,7 +38,7 @@
             style="@style/TextAppearance" />
     </LinearLayout>
 
-    <ListView
+    <com.todoroo.astrid.ui.ErrorCatchingListView
         android:id="@android:id/list"
         android:layout_width="fill_parent"
         android:layout_height="fill_parent"

astrid/res/layout/task_list_body_standard.xml

@@ -22,7 +22,7 @@
 	    style="@style/TextAppearance.TLA_NoItems"/>
     
     <!-- Task List -->
-    <ListView android:id="@android:id/list"           
+    <com.todoroo.astrid.ui.ErrorCatchingListView android:id="@android:id/list"           
         android:layout_width="fill_parent"
         android:layout_height="fill_parent"
         style="@style/ListView" />

astrid/res/layout/widget_config_activity.xml

@@ -17,7 +17,7 @@
         style="TextAppearance.ActionBar_ListsHeader" />
         
     <!-- List -->
-    <ListView android:id="@android:id/list"
+    <com.todoroo.astrid.ui.ErrorCatchingListView android:id="@android:id/list"
         android:layout_width="fill_parent"
         android:layout_height="fill_parent"
         android:layout_weight="100"

astrid/src/com/todoroo/astrid/ui/ErrorCatchingListView.java

@@ -0,0 +1,31 @@
+package com.todoroo.astrid.ui;
+
+import android.content.Context;
+import android.util.AttributeSet;
+import android.view.MotionEvent;
+import android.widget.ListView;
+
+public class ErrorCatchingListView extends ListView {
+
+    public ErrorCatchingListView(Context context) {
+        super(context);
+    }
+
+    public ErrorCatchingListView(Context context, AttributeSet attrs) {
+        super(context, attrs);
+    }
+
+    public ErrorCatchingListView(Context context, AttributeSet attrs, int defStyle) {
+        super(context, attrs, defStyle);
+    }
+
+    @Override
+    public boolean onTouchEvent(MotionEvent ev) {
+        try {
+            return super.onTouchEvent(ev);
+        } catch (Exception e) {
+            return true;
+        }
+    }
+
+}