[android] #5934 - resolves memory leaks on animators + location listeners

2 files changed, 19 insertions, 7 deletions

diff --git a/platform/android/MapboxGLAndroidSDKTestApp/src/main/java/com/mapbox/mapboxsdk/testapp/activity/userlocation/MyLocationTintActivity.java b/platform/android/MapboxGLAndroidSDKTestApp/src/main/java/com/mapbox/mapboxsdk/testapp/activity/userlocation/MyLocationTintActivity.java
index f665b88f60..ba86fa5774 100644
--- a/platform/android/MapboxGLAndroidSDKTestApp/src/main/java/com/mapbox/mapboxsdk/testapp/activity/userlocation/MyLocationTintActivity.java
+++ b/platform/android/MapboxGLAndroidSDKTestApp/src/main/java/com/mapbox/mapboxsdk/testapp/activity/userlocation/MyLocationTintActivity.java
@@ -60,7 +60,7 @@ public class MyLocationTintActivity extends AppCompatActivity implements Locatio
                 final MyLocationViewSettings myLocationViewSettings = mapboxMap.getMyLocationViewSettings();
 
                 // handle default button clicks
-                ViewUtils.attachClickListener(MyLocationTintActivity.this, R.id.default_user_dot_coloring_button,new View.OnClickListener() {
+                ViewUtils.attachClickListener(MyLocationTintActivity.this, R.id.default_user_dot_coloring_button, new View.OnClickListener() {
                     @Override
                     public void onClick(View view) {
                         myLocationViewSettings.setAccuracyTintColor(ContextCompat.getColor(MyLocationTintActivity.this, R.color.my_location_ring));
@@ -69,7 +69,7 @@ public class MyLocationTintActivity extends AppCompatActivity implements Locatio
                 });
 
                 // handle tint user dot button clicks
-                ViewUtils.attachClickListener(MyLocationTintActivity.this, R.id.tint_user_dot_button,new View.OnClickListener() {
+                ViewUtils.attachClickListener(MyLocationTintActivity.this, R.id.tint_user_dot_button, new View.OnClickListener() {
                     @Override
                     public void onClick(View view) {
                         myLocationViewSettings.setAccuracyTintColor(ContextCompat.getColor(MyLocationTintActivity.this, R.color.mapbox_green));
@@ -78,7 +78,7 @@ public class MyLocationTintActivity extends AppCompatActivity implements Locatio
                 });
 
                 // handle tint accuracy ring button clicks
-                ViewUtils.attachClickListener(MyLocationTintActivity.this, R.id.user_accuracy_ring_tint_button,new View.OnClickListener() {
+                ViewUtils.attachClickListener(MyLocationTintActivity.this, R.id.user_accuracy_ring_tint_button, new View.OnClickListener() {
                     @Override
                     public void onClick(View view) {
                         myLocationViewSettings.setAccuracyTintColor(ContextCompat.getColor(MyLocationTintActivity.this, R.color.accent));
@@ -88,7 +88,6 @@ public class MyLocationTintActivity extends AppCompatActivity implements Locatio
             }
         });
 
-        LocationServices.getLocationServices(this).addLocationListener(this);
     }
 
     @Override
@@ -100,6 +99,12 @@ public class MyLocationTintActivity extends AppCompatActivity implements Locatio
     }
 
     @Override
+    protected void onStart() {
+        super.onStart();
+        LocationServices.getLocationServices(this).addLocationListener(this);
+    }
+
+    @Override
     public void onResume() {
         super.onResume();
         mapView.onResume();
@@ -112,6 +117,12 @@ public class MyLocationTintActivity extends AppCompatActivity implements Locatio
     }
 
     @Override
+    protected void onStop() {
+        super.onStop();
+        LocationServices.getLocationServices(this).removeLocationListener(this);
+    }
+
+    @Override
     public void onLowMemory() {
         super.onLowMemory();
         mapView.onLowMemory();
@@ -180,7 +191,7 @@ public class MyLocationTintActivity extends AppCompatActivity implements Locatio
 
         public static void attachClickListener(@NonNull Activity activity, @IdRes int buttonId, @Nullable View.OnClickListener clickListener) {
             View view = activity.findViewById(buttonId);
-            if(view!=null){
+            if (view != null) {
                 view.setOnClickListener(clickListener);
             }
         }
diff --git a/platform/android/MapboxGLAndroidSDKTestApp/src/main/java/com/mapbox/mapboxsdk/testapp/activity/userlocation/MyLocationTrackingModeActivity.java b/platform/android/MapboxGLAndroidSDKTestApp/src/main/java/com/mapbox/mapboxsdk/testapp/activity/userlocation/MyLocationTrackingModeActivity.java
index 273b5134c3..cf0d4c809a 100644
--- a/platform/android/MapboxGLAndroidSDKTestApp/src/main/java/com/mapbox/mapboxsdk/testapp/activity/userlocation/MyLocationTrackingModeActivity.java
+++ b/platform/android/MapboxGLAndroidSDKTestApp/src/main/java/com/mapbox/mapboxsdk/testapp/activity/userlocation/MyLocationTrackingModeActivity.java
@@ -20,12 +20,12 @@ import android.widget.AdapterView;
 import android.widget.ArrayAdapter;
 import android.widget.Spinner;
 import android.widget.Toast;
+
 import com.mapbox.mapboxsdk.camera.CameraUpdateFactory;
 import com.mapbox.mapboxsdk.constants.MapboxConstants;
 import com.mapbox.mapboxsdk.constants.MyBearingTracking;
 import com.mapbox.mapboxsdk.constants.MyLocationTracking;
 import com.mapbox.mapboxsdk.geometry.LatLng;
-import com.mapbox.mapboxsdk.location.LocationServices;
 import com.mapbox.mapboxsdk.maps.MapView;
 import com.mapbox.mapboxsdk.maps.MapboxMap;
 import com.mapbox.mapboxsdk.maps.OnMapReadyCallback;
@@ -145,7 +145,7 @@ public class MyLocationTrackingModeActivity extends AppCompatActivity implements
         }
     }
 
-    private void setInitialPosition(LatLng latLng){
+    private void setInitialPosition(LatLng latLng) {
         mMapboxMap.moveCamera(CameraUpdateFactory.newLatLngZoom(latLng, 14));
         mMapboxMap.setMyLocationEnabled(true);
         mLocationSpinner.setEnabled(true);
@@ -274,6 +274,7 @@ public class MyLocationTrackingModeActivity extends AppCompatActivity implements
                 mMapboxMap.getTrackingSettings().setDismissBearingTrackingOnGesture(state);
                 Toast.makeText(this, "Dismiss bearing mode on gesture = " + state, Toast.LENGTH_SHORT).show();
                 item.setChecked(state);
+                return true;
             default:
                 return super.onOptionsItemSelected(item);
         }