fix an issue where pngrawrastertiles were created read-only even on h…

…ardware which doesn't support supportsAsyncGPUReadback (like android devices) causing error on elevation processing
mapbox · Jul 27, 2021 · 4f4a327 · 4f4a327
1 parent a6c79ae
commit 4f4a327
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/sdkproject/Assets/Mapbox/Core/mapbox-sdk-cs/Map/RawPngRasterTile.cs b/sdkproject/Assets/Mapbox/Core/mapbox-sdk-cs/Map/RawPngRasterTile.cs
@@ -33,7 +33,7 @@ internal override TileResource MakeTileResource(string tilesetId)
 			return TileResource.MakeRawPngRaster(Id, tilesetId);
 		}
 
-		public RawPngRasterTile(CanonicalTileId tileId, string tilesetId) : base(tileId, tilesetId, true)
+		public RawPngRasterTile(CanonicalTileId tileId, string tilesetId, bool useReadonlyTexture = false) : base(tileId, tilesetId, useReadonlyTexture)
 		{
 		}
 	}

diff --git a/sdkproject/Assets/Mapbox/Unity/CustomLayer/MapboxTerrainFactoryManager.cs b/sdkproject/Assets/Mapbox/Unity/CustomLayer/MapboxTerrainFactoryManager.cs
@@ -66,7 +66,14 @@ protected override RasterTile CreateTile(CanonicalTileId tileId, string tilesetI
 			// }
 			// else
 			{
-				rasterTile = new RawPngRasterTile(tileId, tilesetId);
+				if (SystemInfo.supportsAsyncGPUReadback)
+				{
+					rasterTile = new RawPngRasterTile(tileId, tilesetId, true);
+				}
+				else
+				{
+					rasterTile = new RawPngRasterTile(tileId, tilesetId, false);
+				}
 			}
 
 #if UNITY_EDITOR
-Original file line number
+Diff line change
@@ Expand Up @@
     			return TileResource.MakeRawPngRaster(Id, tilesetId);
     		}
-    		public RawPngRasterTile(CanonicalTileId tileId, string tilesetId) : base(tileId, tilesetId, true)
+    		public RawPngRasterTile(CanonicalTileId tileId, string tilesetId, bool useReadonlyTexture = false) : base(tileId, tilesetId, useReadonlyTexture)
     		{
     		}
     	}
@@ Expand Down @@