[android] - move camera logic to dedicated transform class (#6919)

* [android] - move camera logic to dedicated transform class

post camera updates to the message queue, this makes calling an new camera update in the on finish of another update possible. Simplify transform.java implementation.

* fail the step if instrumentation tests fail, fixup typo

1 files changed, 24 insertions, 1 deletions

diff --git a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/NativeMapView.java b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/NativeMapView.java
index e7e3e647af..cdd275ee34 100755
--- a/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/NativeMapView.java
+++ b/platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/NativeMapView.java
@@ -30,6 +30,7 @@ import java.nio.ByteBuffer;
 import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.List;
+import java.util.concurrent.CopyOnWriteArrayList;
 
 // Class that wraps the native methods for convenience
 final class NativeMapView {
@@ -44,6 +45,8 @@ final class NativeMapView {
 
     private final float pixelRatio;
 
+    private CopyOnWriteArrayList<MapView.OnMapChangedListener> onMapChangedListeners;
+
     //
     // Static methods
     //
@@ -83,7 +86,7 @@ final class NativeMapView {
         if (totalMemory < 0) {
             throw new IllegalArgumentException("totalMemory cannot be negative.");
         }
-
+        onMapChangedListeners = new CopyOnWriteArrayList<>();
         this.mapView = mapView;
         nativeMapViewPtr = nativeCreate(cachePath, dataPath, apkPath, pixelRatio, availableProcessors, totalMemory);
     }
@@ -778,4 +781,24 @@ final class NativeMapView {
     private native Feature[] nativeQueryRenderedFeaturesForBox(long nativeMapViewPtr, float left, float top, float right, float bottom, String[] layerIds);
 
     private native void nativeSetAPIBaseURL(long nativeMapViewPtr, String baseUrl);
+
+    //
+    // MapChangeEvents
+    //
+
+    void addOnMapChangedListener(@NonNull MapView.OnMapChangedListener listener) {
+        onMapChangedListeners.add(listener);
+    }
+
+    void removeOnMapChangedListener(@NonNull MapView.OnMapChangedListener listener) {
+        onMapChangedListeners.remove(listener);
+    }
+
+    void onMapChangedEventDispatch(int mapChange) {
+        if (onMapChangedListeners != null) {
+            for (MapView.OnMapChangedListener onMapChangedListener : onMapChangedListeners) {
+                onMapChangedListener.onMapChanged(mapChange);
+            }
+        }
+    }
 }