[android] #5276 - icon should be optional for MarkerView

platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/annotations/IconFactory.java

@@ -33,6 +33,7 @@ public final class IconFactory {
     private Context mContext;
     private static IconFactory sInstance;
     private Icon mDefaultMarker;
+    private Icon mDefaultMarkerView;
     private BitmapFactory.Options mOptions;
 
     private int mNextId = 0;
@@ -121,6 +122,13 @@ public Icon defaultMarker() {
         return mDefaultMarker;
     }
 
+    public Icon defaultMarkerView() {
+        if (mDefaultMarkerView == null) {
+            mDefaultMarkerView = fromResource(R.drawable.default_markerview);
+        }
+        return mDefaultMarkerView;
+    }
+
     private Icon fromInputStream(@NonNull InputStream is) {
         Bitmap bitmap = BitmapFactory.decodeStream(is, null, mOptions);
         return fromBitmap(bitmap);

platform/android/MapboxGLAndroidSDK/src/main/java/com/mapbox/mapboxsdk/maps/MapboxMap.java

@@ -20,6 +20,7 @@
 import com.mapbox.mapboxsdk.annotations.BaseMarkerOptions;
 import com.mapbox.mapboxsdk.annotations.BaseMarkerViewOptions;
 import com.mapbox.mapboxsdk.annotations.Icon;
+import com.mapbox.mapboxsdk.annotations.IconFactory;
 import com.mapbox.mapboxsdk.annotations.InfoWindow;
 import com.mapbox.mapboxsdk.annotations.Marker;
 import com.mapbox.mapboxsdk.annotations.MarkerOptions;
@@ -1208,7 +1209,12 @@ private Marker prepareMarker(BaseMarkerOptions markerOptions) {
 
     private MarkerView prepareViewMarker(BaseMarkerViewOptions markerViewOptions) {
         MarkerView marker = markerViewOptions.getMarker();
-        marker.setIcon(markerViewOptions.getIcon());
+
+        Icon icon = markerViewOptions.getIcon();
+        if (icon == null) {
+            icon = IconFactory.getInstance(mMapView.getContext()).defaultMarkerView();
+        }
+        marker.setIcon(icon);
         return marker;
     }