Don't consider gtasks sync provider 'active' if astrid logged in

astrid/plugin-src/com/todoroo/astrid/gtasks/sync/GtasksSyncV2Provider.java

@@ -108,7 +108,7 @@ public class GtasksSyncV2Provider extends SyncV2Provider {
 
     @Override
     public boolean isActive() {
-        return gtasksPreferenceService.isLoggedIn();
+        return gtasksPreferenceService.isLoggedIn() && !actFmPreferenceService.isLoggedIn();
     }
 
     @Override

astrid/src/com/todoroo/astrid/service/SyncV2Service.java

@@ -68,19 +68,8 @@ public class SyncV2Service {
         if (active.size() == 0)
             return false;
 
-        if (active.size() > 1) {
+        if (active.size() > 1) { // This should never happen anymore--they can't be active at the same time, but if for some reason they both are, just use ActFm
-            SyncResultCallback newCallback = new WidgetUpdatingCallbackWrapper(callback) {
+            active.get(1).synchronizeActiveTasks(manual, new WidgetUpdatingCallbackWrapper(callback));
-                private int next = 1;
-
-                @Override
-                public void finished() {
-                    super.finished();
-                    if (next < active.size())
-                        active.get(next++).synchronizeActiveTasks(manual, this);
-                }
-            };
-
-            active.get(0).synchronizeActiveTasks(manual, newCallback);
         } else if (active.size() == 1) {
             active.get(0).synchronizeActiveTasks(manual, new WidgetUpdatingCallbackWrapper(callback));
         }