Second attempt at proper task startup.

MyTracks/src/com/google/android/apps/mytracks/services/PeriodicTaskExecuter.java

@@ -44,6 +44,8 @@ public class PeriodicTaskExecuter {
                               TrackRecordingService service) {
     this.task = task;
     this.service = service;
+
+    task.start();
   }
 
   /**

MyTracks/src/com/google/android/apps/mytracks/services/SplitManager.java

@@ -133,7 +133,7 @@ public class SplitManager {
     if (!service.isRecording()) {
       return;
     }
-    
+
     if (splitFrequency < 1) {
       if (splitExecuter != null) {
         splitExecuter.shutdown();
@@ -143,7 +143,6 @@ public class SplitManager {
     if (splitFrequency > 0) {
       if (splitExecuter == null) {
         TimeSplitTask splitter = new TimeSplitTask();
-        splitter.start();
         splitExecuter = new PeriodicTaskExecuter(splitter, service);
       }
       splitExecuter.scheduleTask(splitFrequency * 60000);

MyTracks/src/com/google/android/apps/mytracks/services/TaskExecuterManager.java

@@ -61,7 +61,6 @@ public class TaskExecuterManager {
       }
     } else {
       if (executer == null) {
-        task.start();
         executer = new PeriodicTaskExecuter(task, service);
       }
       executer.scheduleTask(frequency * 60000);
@@ -73,11 +72,10 @@ public class TaskExecuterManager {
    */
   public void restore() {
     if (frequency > 0) {
-      task.start();
       executer.scheduleTask(frequency * 60000);
     }
   }
-  
+
   /**
    * Shuts down this executer.
    */

MyTracks/src/com/google/android/apps/mytracks/services/TrackRecordingService.java

@@ -681,7 +681,6 @@ public class TrackRecordingService extends Service implements LocationListener {
             }
 
             // TODO: Either use TaskExecuterManager everywhere, or get rid of it
-            announcer.start();
             announcementExecuter = new PeriodicTaskExecuter(announcer,
                 TrackRecordingService.this);
           }