Fixing issue 35 - detecting errors during "export all".

MyTracks/src/com/google/android/apps/mytracks/ExportAllTracks.java

@@ -17,6 +17,7 @@ package com.google.android.apps.mytracks;
 
 import com.google.android.apps.mytracks.content.MyTracksProviderUtils;
 import com.google.android.apps.mytracks.content.TracksColumns;
+import com.google.android.apps.mytracks.io.TrackWriter;
 import com.google.android.apps.mytracks.io.TrackWriterFactory;
 import com.google.android.apps.mytracks.io.TrackWriterFactory.TrackFileFormat;
 import com.google.android.maps.mytracks.R;
@@ -154,6 +155,7 @@ public class ExportAllTracks {
       if (cursor == null) {
         return;
       }
+
       final int trackCount = cursor.getCount();
       Log.i(MyTracksConstants.TAG,
           "ExportAllTracks: Exporting: " + cursor.getCount() + " tracks.");
@@ -161,7 +163,9 @@ public class ExportAllTracks {
       MyTracks.getInstance().runOnUiThread(new Runnable() {
         public void run() {
           makeProgressDialog(trackCount);
-        }});
+        }
+      });
+
       for (int i = 0; cursor.moveToNext(); i++) {
         final int status = i;
         MyTracks.getInstance().runOnUiThread(new Runnable() {
@@ -177,16 +181,27 @@ public class ExportAllTracks {
 
         long id = cursor.getLong(idxTrackId);
         Log.i(MyTracksConstants.TAG, "ExportAllTracks: exporting: " + id);
-        TrackWriterFactory.newWriter(context, providerUtils, id, format)
+        TrackWriter writer =
-            .writeTrack();
+            TrackWriterFactory.newWriter(context, providerUtils, id, format);
+        writer.writeTrack();
+
+        if (!writer.wasSuccess()) {
+          // Abort the whole export on the first error.
+          int error = writer.getErrorMessage();
+          Toast.makeText(context, error, Toast.LENGTH_LONG).show();
+          return;
+        }
       }
     } finally {
       if (cursor != null) {
         cursor.close();
       }
-    }
+      if (progress != null) {
-    if (progress != null) {
+        synchronized (this) {
-      progress.dismiss();
+          progress.dismiss();
+          progress = null;
+        }
+      }
     }
   }