In MyTracksMapFragment, remove dependency on onSelectedTrackChanged.

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

@@ -470,6 +470,13 @@ public class TrackDetailActivity extends AbstractSendToGoogleActivity
     return trackDataHub;
   }
 
+  /**
+   * Gets the track id.
+   */
+  public long getTrackId() {
+    return trackId;
+  }
+  
   /**
    * Gets the marker id.
    */

MyTracks/src/com/google/android/apps/mytracks/fragments/MyTracksMapFragment.java

@@ -62,6 +62,7 @@ import android.view.MenuItem;
 import android.view.View;
 import android.view.View.OnClickListener;
 import android.view.ViewGroup;
+import android.view.ViewTreeObserver.OnGlobalLayoutListener;
 import android.widget.ImageButton;
 import android.widget.RelativeLayout;
 import android.widget.TextView;
@@ -93,8 +94,9 @@ public class MyTracksMapFragment extends SupportMapFragment implements TrackData
   private static final int MAP_VIEW_PADDING = 32;
 
   // States from TrackDetailActivity, set in onResume
-  private TrackDataHub trackDataHub;
+  private long trackId;
   private long markerId;
+  private TrackDataHub trackDataHub;
   
   // Current location
   private Location currentLocation;
@@ -248,8 +250,6 @@ public class MyTracksMapFragment extends SupportMapFragment implements TrackData
           }
         }
       });
-      googleMap.moveCamera(
-          CameraUpdateFactory.newLatLngZoom(getDefaultLatLng(), googleMap.getMinZoomLevel()));
     }
   }
 
@@ -258,6 +258,7 @@ public class MyTracksMapFragment extends SupportMapFragment implements TrackData
     super.onResume();
 
     // First obtain the states from TrackDetailActivity
+    trackId = ((TrackDetailActivity) getActivity()).getTrackId();
     markerId = ((TrackDetailActivity) getActivity()).getMarkerId();
     resumeTrackDataHub();
 
@@ -281,12 +282,15 @@ public class MyTracksMapFragment extends SupportMapFragment implements TrackData
     // setWarningMessage depends on resumeTrackDataHub being invoked beforehand
     setWarningMessage(isGpsProviderEnabled);
     
-    if (markerId == -1L && isInTrackingMode()) {
+    currentTrack = MyTracksProviderUtils.Factory.get(getActivity()).getTrack(trackId);
-      updateCurrentLocation(true);      
+    mapOverlay.setShowEndMarker(!isSelectedTrackRecording());
+    if (markerId != -1L) {
+      showMarker(markerId);
     } else {
-      if (googleMap != null) {
+      if (keepCurrentLocationVisible && currentLocation != null && isSelectedTrackRecording()) {
-        googleMap.moveCamera(
+        updateCurrentLocation(true);
-            CameraUpdateFactory.newLatLngZoom(getDefaultLatLng(), googleMap.getMinZoomLevel()));
+      } else {
+        showTrack();
       }      
     }   
   }
@@ -374,22 +378,12 @@ public class MyTracksMapFragment extends SupportMapFragment implements TrackData
   
   @Override
   public void onSelectedTrackChanged(final Track track) {
-    if (isResumed()) {
+    // We don't care.
-      currentTrack = track;
-      mapOverlay.setShowEndMarker(!isSelectedTrackRecording());
-      if (markerId != -1L) {
-        showMarker(markerId);
-      } else {
-        if (!isInTrackingMode()) {
-          showTrack();
-        }
-      }
-    }
   }
   
   @Override
   public void onTrackUpdated(Track track) {
-    // We don't care.
+    currentTrack = track;
   }
 
   @Override
@@ -509,7 +503,7 @@ public class MyTracksMapFragment extends SupportMapFragment implements TrackData
    */
   private synchronized void resumeTrackDataHub() {
     trackDataHub = ((TrackDetailActivity) getActivity()).getTrackDataHub();
-    trackDataHub.registerTrackDataListener(this, EnumSet.of(TrackDataType.SELECTED_TRACK,
+    trackDataHub.registerTrackDataListener(this, EnumSet.of(TrackDataType.TRACKS_TABLE,
         TrackDataType.WAYPOINTS_TABLE, TrackDataType.SAMPLED_IN_TRACK_POINTS_TABLE,
         TrackDataType.SAMPLED_OUT_TRACK_POINTS_TABLE, TrackDataType.PREFERENCE));
   }
@@ -571,53 +565,83 @@ public class MyTracksMapFragment extends SupportMapFragment implements TrackData
   }
 
   /**
-   * Sets the camera over a track.
+   * Shows the current track by moving the camera over the track.
    */
   private void showTrack() {
-    getActivity().runOnUiThread(new Runnable() {
+    if (googleMap == null || currentTrack == null) {
-        @Override
+      return;
-      public void run() {
+    }
-        if (!isResumed() || googleMap == null || currentTrack == null
+    if (currentTrack.getNumberOfPoints() < 2) {
-            || currentTrack.getNumberOfPoints() < 2) {
+      googleMap.moveCamera(
-          return;
+          CameraUpdateFactory.newLatLngZoom(getDefaultLatLng(), googleMap.getMinZoomLevel()));
-        }
+      return;
+    }
 
-        /**
+    if (mapView == null) {
-         * Check that mapView is valid.
+      return;
-         */
+    }
-        if (mapView == null || mapView.getWidth() == 0 || mapView.getHeight() == 0) {
-          return;
-        }
     
-        TripStatistics tripStatistics = currentTrack.getTripStatistics();
+    if (mapView.getWidth() == 0 || mapView.getHeight() == 0) {
-        int latitudeSpanE6 = tripStatistics.getTop() - tripStatistics.getBottom();
+      if (mapView.getViewTreeObserver().isAlive()) {
-        int longitudeSpanE6 = tripStatistics.getRight() - tripStatistics.getLeft();
+        mapView.getViewTreeObserver().addOnGlobalLayoutListener(new OnGlobalLayoutListener() {
-        if (latitudeSpanE6 > 0 && latitudeSpanE6 < 180E6 && longitudeSpanE6 > 0
+            @Override
-            && longitudeSpanE6 < 360E6) {
+          public void onGlobalLayout() {
-          LatLng southWest = new LatLng(
+            ApiAdapterFactory.getApiAdapter()
-              tripStatistics.getBottomDegrees(), tripStatistics.getLeftDegrees());
+                .removeGlobalLayoutListener(mapView.getViewTreeObserver(), this);
-          LatLng northEast = new LatLng(
+            getActivity().runOnUiThread(new Runnable() {
-              tripStatistics.getTopDegrees(), tripStatistics.getRightDegrees());
+                @Override
-          LatLngBounds bounds = LatLngBounds.builder()
+              public void run() {
-              .include(southWest).include(northEast).build();
+                if (isResumed()) {
-
+                  moveCameraOverTrack();
-          /**
+                }
-           * Note cannot call CameraUpdateFactory.newLatLngBounds(LatLngBounds bounds,
+              }
-           * int padding) if the map view has not undergone layout. Thus calling
+            });
-           * CameraUpdateFactory.newLatLngBounds(LatLngBounds bounds, int width, int
+          }
-           * height, int padding) after making sure that mapView is valid in the
+        });
-           * above code.
-           */
-          CameraUpdate cameraUpdate = CameraUpdateFactory.newLatLngBounds(
-              bounds, mapView.getWidth(), mapView.getHeight(), MAP_VIEW_PADDING);
-          googleMap.moveCamera(cameraUpdate);
-        }
       }
-    });
+      return;
+    }
+    moveCameraOverTrack();
   }
 
   /**
-   * Sets the camera over a marker.
+   * Moves the camera over the current track.
+   */
+  private void moveCameraOverTrack() {
+    /**
+     * Check all the required variables.
+     */
+    if (googleMap == null || currentTrack == null || currentTrack.getNumberOfPoints() < 2
+        || mapView == null || mapView.getWidth() == 0 || mapView.getHeight() == 0) {
+      return;
+    }
+
+    TripStatistics tripStatistics = currentTrack.getTripStatistics();
+    int latitudeSpanE6 = tripStatistics.getTop() - tripStatistics.getBottom();
+    int longitudeSpanE6 = tripStatistics.getRight() - tripStatistics.getLeft();
+    if (latitudeSpanE6 > 0 && latitudeSpanE6 < 180E6 && longitudeSpanE6 > 0
+        && longitudeSpanE6 < 360E6) {
+      LatLng southWest = new LatLng(
+          tripStatistics.getBottomDegrees(), tripStatistics.getLeftDegrees());
+      LatLng northEast = new LatLng(
+          tripStatistics.getTopDegrees(), tripStatistics.getRightDegrees());
+      LatLngBounds bounds = LatLngBounds.builder().include(southWest).include(northEast).build();
+
+      /**
+       * Note cannot call CameraUpdateFactory.newLatLngBounds(LatLngBounds
+       * bounds, int padding) if the map view has not undergone layout. Thus
+       * calling CameraUpdateFactory.newLatLngBounds(LatLngBounds bounds, int
+       * width, int height, int padding) after making sure that mapView is valid
+       * in the above code.
+       */
+      CameraUpdate cameraUpdate = CameraUpdateFactory.newLatLngBounds(
+          bounds, mapView.getWidth(), mapView.getHeight(), MAP_VIEW_PADDING);
+      googleMap.moveCamera(cameraUpdate);
+    }
+  }
+
+  /**
+   * Shows a marker by moving the camera over the marker.
    * 
    * @param id the marker id
    */
@@ -653,13 +677,6 @@ public class MyTracksMapFragment extends SupportMapFragment implements TrackData
     return new LatLng(DEFAULT_LATITUDE, DEFAULT_LONGITUDE);
   }
 
-  /**
-   * Returns true if in tracking mode.
-   */
-  private boolean isInTrackingMode() {
-    return keepCurrentLocationVisible && currentLocation != null && isSelectedTrackRecording();
-  }
-  
   /**
    * Returns true if the location is visible. Needs to run on the UI thread.
    * 

MyTracks/src/com/google/android/apps/mytracks/util/Api16Adapter.java

@@ -21,6 +21,8 @@ import com.google.android.apps.mytracks.widgets.TrackWidgetProvider;
 import android.annotation.TargetApi;
 import android.appwidget.AppWidgetManager;
 import android.os.Bundle;
+import android.view.ViewTreeObserver;
+import android.view.ViewTreeObserver.OnGlobalLayoutListener;
 
 /**
  * API level 16 specific implementation of the {@link ApiAdapter}.
@@ -49,4 +51,10 @@ public class Api16Adapter extends Api14Adapter {
     bundle.putInt(APP_WIDGET_SIZE_KEY, size);
     appWidgetManager.updateAppWidgetOptions(appWidgetId, bundle);
   }
+  
+  @Override
+  public void removeGlobalLayoutListener(
+      ViewTreeObserver observer, OnGlobalLayoutListener listener) {
+    observer.removeOnGlobalLayoutListener(listener);
+  }
 }

MyTracks/src/com/google/android/apps/mytracks/util/Api8Adapter.java

@@ -30,6 +30,8 @@ import android.bluetooth.BluetoothSocket;
 import android.content.SharedPreferences.Editor;
 import android.util.Log;
 import android.view.MenuItem;
+import android.view.ViewTreeObserver;
+import android.view.ViewTreeObserver.OnGlobalLayoutListener;
 import android.view.Window;
 import android.widget.ArrayAdapter;
 import android.widget.ListView;
@@ -155,4 +157,11 @@ public class Api8Adapter implements ApiAdapter {
   public void setAppWidgetSize(AppWidgetManager appWidgetManager, int appWidgetId, int size) {
     // Do nothing    
   }
+
+  @SuppressWarnings("deprecation")
+  @Override
+  public void removeGlobalLayoutListener(
+      ViewTreeObserver observer, OnGlobalLayoutListener listener) {
+    observer.removeGlobalOnLayoutListener(listener);    
+  }
 }

MyTracks/src/com/google/android/apps/mytracks/util/ApiAdapter.java

@@ -25,6 +25,7 @@ import android.bluetooth.BluetoothDevice;
 import android.bluetooth.BluetoothSocket;
 import android.content.SharedPreferences;
 import android.view.MenuItem;
+import android.view.ViewTreeObserver;
 import android.widget.ArrayAdapter;
 import android.widget.ListView;
 
@@ -195,10 +196,23 @@ public interface ApiAdapter {
   
   /**
    * Sets the app widget size.
+   * <p> 
+   * Due to changes in API level 16.
    * 
    * @param appWidgetManager the app widget manager.
    * @param appWidgetId the app widgit id
    * @param size the size
    */
   public void setAppWidgetSize(AppWidgetManager appWidgetManager, int appWidgetId, int size);
+  
+  /**
+   * Removes the global layout listener.
+   * <p>
+   * Due to changes in API level 16.
+   * 
+   * @param observer the observer
+   * @param listener the listener
+   */
+  public void removeGlobalLayoutListener(
+      ViewTreeObserver observer, ViewTreeObserver.OnGlobalLayoutListener listener);
 }