fix location engines activation calling on connected method if the location engine was already connected (#587)

Author: Guardiola31337

mapbox/libandroid-telemetry/src/main/java/com/mapbox/services/android/telemetry/location/GoogleLocationEngine.java

@@ -48,9 +48,7 @@ public static synchronized LocationEngine getLocationEngine(Context context) {
 
   @Override
   public void activate() {
-    if (googleApiClient != null && !googleApiClient.isConnected()) {
-      googleApiClient.connect();
-    }
+    connect();
   }
 
   @Override
@@ -140,4 +138,14 @@ public void onLocationChanged(Location location) {
       listener.onLocationChanged(location);
     }
   }
+
+  private void connect() {
+    if (googleApiClient != null) {
+      if (googleApiClient.isConnected()) {
+        onConnected(null);
+      } else {
+        googleApiClient.connect();
+      }
+    }
+  }
 }

mapbox/libandroid-telemetry/src/main/java/com/mapbox/services/android/telemetry/location/LostLocationEngine.java

@@ -47,9 +47,7 @@ public static synchronized LocationEngine getLocationEngine(Context context) {
    */
   @Override
   public void activate() {
-    if (lostApiClient != null && !lostApiClient.isConnected()) {
-      lostApiClient.connect();
-    }
+    connect();
   }
 
   /**
@@ -167,4 +165,14 @@ public void onLocationChanged(Location location) {
       listener.onLocationChanged(location);
     }
   }
+
+  private void connect() {
+    if (lostApiClient != null) {
+      if (lostApiClient.isConnected()) {
+        onConnected();
+      } else {
+        lostApiClient.connect();
+      }
+    }
+  }
 }