Cherrypick for release-xoai (#477)

* [docs] update changelog for release-xoai final

* Set gps bearing immediately while change to gps mode (#470)

Co-authored-by: kevin <kevin.li@mapbox.com>

Author: tobrun

CHANGELOG.md

@@ -2,12 +2,7 @@
 
 Mapbox welcomes participation and contributions from everyone. Please read [`Contributing Guide`](https://github.com/mapbox/mapbox-gl-native/blob/master/CONTRIBUTING.md) to get started.
 
-## 9.3.0-beta.1 - July 9, 2020
-
- - Updated Mapbox GL Native library to v1.8.1. ([#464](https://github.com/mapbox/mapbox-gl-native-android/pull/464))
- - Not change file source status while changing cache path ([#457](https://github.com/mapbox/mapbox-gl-native-android/pull/457))
-
-## 9.3.0-alpha.1 - July 2, 2020
+## 9.3.0 - July 15, 2020
 Starting this release, the Mapbox Maps SDK for Android will use a pre-built GL Native binary licensed under the [Mapbox Terms of Service](https://www.mapbox.com/legal/tos). The license of the Maps SDK for Android remains BSD-2. For more information on using this and future releases, see [README.md](https://github.com/mapbox/mapbox-gl-native-android/blob/master/README.md). For more details on licensing, see [LICENSE.md](https://github.com/mapbox/mapbox-gl-native-android/blob/master/LICENSE.md).
 
 ### Improvements and bug fixes
@@ -16,13 +11,16 @@ Starting this release, the Mapbox Maps SDK for Android will use a pre-built GL N
  - Clarified the error message when an HTTP request has been cancelled. ([#433](https://github.com/mapbox/mapbox-gl-native-android/pull/433))
  - Fixed a memory leak that would occur when a source was removed. ([#412](https://github.com/mapbox/mapbox-gl-native-android/pull/412))
  - Fixed a bug that would halt the application during a scheduled snapshot when the map renderer was stopped. ([#390](https://github.com/mapbox/mapbox-gl-native-android/pull/390)
+ - Not change file source status while changing cache path ([#457](https://github.com/mapbox/mapbox-gl-native-android/pull/457))
+ - Avoid null pointer exception when the underlying hardware is too slow to create the surface ([#471](https://github.com/mapbox/mapbox-gl-native-android/pull/471))
 
 ### User location indicator
  - Double-tapping the map when the camera is tracking the user location indicator now zooms to the `LocationComponent`'s location. ([#378](https://github.com/mapbox/mapbox-gl-native-android/pull/378))
  - Improved camera rotation to use the shortest path. ([#426](https://github.com/mapbox/mapbox-gl-native-android/pull/426))
  - Added support for using a background image on GPS mode when `LocationComponentActivationOptions#useSpecializedLocationLayer` is activated . ([#439](https://github.com/mapbox/mapbox-gl-native-android/pull/439))
  - Fixed a bug where an in-progress animation of an updating `LocationComponent` would be interrupted when a style was changed. ([#437](https://github.com/mapbox/mapbox-gl-native-android/pull/437))
  - Updated documentation to reflect that the `FOREGROUND_LAYER` constant is usable for relative positioning in both normal and specialized rendering modes. [#413](https://github.com/mapbox/mapbox-gl-native-android/pull/413)
+ - Set gps bearing immediately while changing to gps mode. ([#470](https://github.com/mapbox/mapbox-gl-native-android/pull/470))
 
 ### Other changes
  - Updated javadoc documentation for `removeSource`/`removeLayer`/`removeLayerAt`. ([#422](https://github.com/mapbox/mapbox-gl-native-android/pull/422))

MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/location/LocationAnimatorCoordinator.java

@@ -482,6 +482,11 @@ void cancelTiltAnimation() {
     cancelAnimator(ANIMATOR_TILT);
   }
 
+  void cancelAndRemoveGpsBearingAnimation() {
+    cancelAnimator(ANIMATOR_LAYER_GPS_BEARING);
+    animatorArray.remove(ANIMATOR_LAYER_GPS_BEARING);
+  }
+
   /**
    * Cancel the pulsing circle location animator.
    */

MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/location/LocationComponent.java

@@ -48,6 +48,7 @@
 import static com.mapbox.mapboxsdk.location.LocationComponentConstants.DEFAULT_TRACKING_TILT_ANIM_DURATION;
 import static com.mapbox.mapboxsdk.location.LocationComponentConstants.DEFAULT_TRACKING_ZOOM_ANIM_DURATION;
 import static com.mapbox.mapboxsdk.location.LocationComponentConstants.TRANSITION_ANIMATION_DURATION_MS;
+import static com.mapbox.mapboxsdk.location.modes.RenderMode.GPS;
 
 /**
  * The Location Component provides location awareness to your mobile application. Enabling this
@@ -673,6 +674,10 @@ public int getCameraMode() {
    */
   public void setRenderMode(@RenderMode.Mode int renderMode) {
     checkActivationState();
+    if (lastLocation != null && renderMode == GPS) {
+      locationAnimatorCoordinator.cancelAndRemoveGpsBearingAnimation();
+      locationLayerController.setGpsBearing(lastLocation.getBearing());
+    }
     locationLayerController.setRenderMode(renderMode);
     updateLayerOffsets(true);
     updateCompassListenerState(true);

MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/location/LocationLayerController.java

@@ -106,6 +106,10 @@ void applyStyle(@NonNull LocationComponentOptions options) {
     }
   }
 
+  void setGpsBearing(float gpsBearing) {
+    locationLayerRenderer.setGpsBearing(gpsBearing);
+  }
+
   void setRenderMode(@RenderMode.Mode int renderMode) {
     if (this.renderMode == renderMode) {
       return;

MapboxGLAndroidSDK/src/test/java/com/mapbox/mapboxsdk/location/LocationAnimatorCoordinatorTest.kt

@@ -677,6 +677,15 @@ class LocationAnimatorCoordinatorTest {
     verify { animatorProvider.floatAnimator(any(), any(), 5) }
   }
 
+  @Test
+  fun remove_gps_animator() {
+    val animator = mockk<MapboxFloatAnimator>(relaxed = true)
+    locationAnimatorCoordinator.animatorArray.put(ANIMATOR_LAYER_GPS_BEARING, animator)
+
+    locationAnimatorCoordinator.cancelAndRemoveGpsBearingAnimation()
+    assertTrue(locationAnimatorCoordinator.animatorArray.get(ANIMATOR_LAYER_GPS_BEARING) == null)
+  }
+
   private fun getListenerHoldersSet(vararg animatorTypes: Int): Set<AnimatorListenerHolder> {
     return HashSet<AnimatorListenerHolder>().also {
       for (type in animatorTypes) {

MapboxGLAndroidSDK/src/test/java/com/mapbox/mapboxsdk/location/LocationComponentTest.kt

@@ -469,6 +469,34 @@ class LocationComponentTest {
     verify(renderChangeListener).onRenderModeChanged(RenderMode.NORMAL)
   }
 
+  @Test
+  fun change_to_gps_mode_symbolLayerBearingValue() {
+    val location = Location("test")
+    location.bearing = 50f
+    val projection: Projection = mock(Projection::class.java)
+    `when`(projection.getMetersPerPixelAtLatitude(location.latitude)).thenReturn(10.0)
+    `when`(mapboxMap.projection).thenReturn(projection)
+    `when`(style.isFullyLoaded).thenReturn(true)
+    `when`(mapboxMap.cameraPosition).thenReturn(CameraPosition.DEFAULT)
+
+    locationComponent.activateLocationComponent(
+      LocationComponentActivationOptions.builder(context, style)
+        .locationComponentOptions(locationComponentOptions)
+        .useDefaultLocationEngine(false)
+        .build()
+    )
+    locationComponent.isLocationComponentEnabled = true
+    locationComponent.onStart()
+    locationComponent.renderMode = RenderMode.NORMAL
+    locationComponent.forceLocationUpdate(location)
+
+    verify(locationLayerController, times(0)).setGpsBearing(50f)
+
+    locationComponent.renderMode = RenderMode.GPS
+    verify(locationLayerController, times(1)).setGpsBearing(50f)
+    verify(locationAnimatorCoordinator).cancelAndRemoveGpsBearingAnimation()
+  }
+
   @Test
   fun tiltWhileTracking_notReady() {
     `when`(mapboxMap.cameraPosition).thenReturn(CameraPosition.DEFAULT)