fix: prevent NPE in LatLngBounds.contains by adding null checks for positions (#1675)

Author: kikoso

library/src/main/java/com/google/maps/android/clustering/view/ClusterRendererMultipleItems.java

@@ -524,7 +524,7 @@ public void run() {
             }
 
             for (final MarkerWithPosition marker : markersToRemove) {
-                boolean onScreen = visibleBounds.contains(marker.position);
+                boolean onScreen = marker.position != null && visibleBounds.contains(marker.position);
                 if (onScreen && mAnimate) {
                     final Point point = mSphericalMercatorProjection.toPoint(marker.position);
                     final Point closest = findClosestCluster(newClustersOnScreen, point);

library/src/main/java/com/google/maps/android/clustering/view/DefaultAdvancedMarkersClusterRenderer.java

@@ -453,7 +453,7 @@ public void run() {
             if (DefaultAdvancedMarkersClusterRenderer.this.mClusters != null && mAnimate) {
                 existingClustersOnScreen = new ArrayList<>();
                 for (Cluster<T> c : DefaultAdvancedMarkersClusterRenderer.this.mClusters) {
-                    if (shouldRenderAsCluster(c) && visibleBounds.contains(c.getPosition())) {
+                    if (shouldRenderAsCluster(c) && c.getPosition() != null && visibleBounds.contains(c.getPosition())) {
                         Point point = mSphericalMercatorProjection.toPoint(c.getPosition());
                         existingClustersOnScreen.add(point);
                     }
@@ -492,7 +492,7 @@ public void run() {
             if (mAnimate) {
                 newClustersOnScreen = new ArrayList<>();
                 for (Cluster<T> c : clusters) {
-                    if (shouldRenderAsCluster(c) && visibleBounds.contains(c.getPosition())) {
+                    if (shouldRenderAsCluster(c) && c.getPosition() != null && visibleBounds.contains(c.getPosition())) {
                         Point p = mSphericalMercatorProjection.toPoint(c.getPosition());
                         newClustersOnScreen.add(p);
                     }

library/src/main/java/com/google/maps/android/clustering/view/DefaultClusterRenderer.java

@@ -492,7 +492,7 @@ public void run() {
 
             // Remove the old markers, animating them into clusters if zooming out.
             for (final MarkerWithPosition marker : markersToRemove) {
-                boolean onScreen = visibleBounds.contains(marker.position);
+                boolean onScreen = marker.position != null && visibleBounds.contains(marker.position);
                 // Don't animate when zooming out more than 3 zoom levels.
                 // TODO: drop animation based on speed of device & number of markers to animate.
                 if (!zoomingIn && zoomDelta > -3 && onScreen && mAnimate) {