Avoid setting camera for Anchor animator (#6628)

Currently we call set camera per each Animator independently.
For Anchor animator it doesn't make sense to call set camera since it
will do nothing.

This PR makes that more explicit and avoids some minor calls for the
Anchor animator.

Moreover, this PR also resets the camera builder instead of creating a
new one on every animator update.

cc @mapbox/maps-android

GitOrigin-RevId: 2edc883a9b63a7cbfe8875f84fc1cdbb0364ff78

Author: jush

plugin-animation/src/main/java/com/mapbox/maps/plugin/animation/CameraAnimationsExt.kt

@@ -340,11 +340,11 @@ internal fun updateCameraValue(
  * Convenience method to clear camera options so it can be reused instead of creating
  * new [CameraOptions.Builder].
  */
-private fun CameraOptions.Builder.clear() {
+internal fun CameraOptions.Builder.clear() {
   center(null)
-    .padding(null)
-    .anchor(null)
-    .zoom(null)
-    .bearing(null)
-    .pitch(null)
+  padding(null)
+  anchor(null)
+  zoom(null)
+  bearing(null)
+  pitch(null)
 }

plugin-animation/src/main/java/com/mapbox/maps/plugin/animation/CameraAnimationsPluginImpl.kt

@@ -377,9 +377,6 @@ internal class CameraAnimationsPluginImpl : CameraAnimationsPlugin, MapCameraPlu
                 AnimationFinishStatus.ENDED -> it.onAnimatorEnding(type, this, owner)
               }
             }
-            if (runningAnimatorsQueue.isEmpty()) {
-              commitChanges()
-            }
           } ?: throw MapboxCameraAnimationException(
             "Could not finish animation as it must be an instance of CameraAnimator and not null!"
           )
@@ -399,22 +396,24 @@ internal class CameraAnimationsPluginImpl : CameraAnimationsPlugin, MapCameraPlu
     // add current animator to queue-set if was not present
     runningAnimatorsQueue.add(animator)
 
-    // set current animator value in any case
-    updateCameraValue(animator, animator.animatedValue, cameraOptionsBuilder)
-
+    // Anchor animator is a special case. It won't affect the camera by itself so we cache the value
+    // and apply it to the other animators.
     if (animator.type == CameraAnimatorType.ANCHOR) {
       anchor = animator.animatedValue as ScreenCoordinate
-    }
+    } else {
+      // set current animator value to the camera options builder
+      updateCameraValue(animator, animator.animatedValue, cameraOptionsBuilder)
 
-    // commit applies changes immediately
-    // this helps to avoid camera animations jitter noticeable on high zoom levels using location puck following mode.
-    commitChanges()
+      // commit applies changes immediately
+      // this helps to avoid camera animations jitter noticeable on high zoom levels using location puck following mode.
+      commitChanges()
+    }
   }
 
   private fun commitChanges() {
     performMapJump(cameraOptionsBuilder.anchor(anchor).build())
-    // reset values
-    cameraOptionsBuilder = CameraOptions.Builder()
+    // reset values so it can be reused
+    cameraOptionsBuilder.clear()
   }
 
   private fun cancelAnimatorSet() {