[ios, macos] Initialize offline storage on main thread before snapsho…

…tting
mapbox · Apr 7, 2020 · 17ad571 · 17ad571
1 parent 928f37f
commit 17ad571
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/platform/darwin/src/MGLMapSnapshotter.mm b/platform/darwin/src/MGLMapSnapshotter.mm
@@ -301,7 +301,12 @@ - (void)startWithQueue:(dispatch_queue_t)queue overlayHandler:(MGLMapSnapshotOve
     if (!completion) {
         return;
     }
-    if (!mbgl::Scheduler::GetCurrent()) {
+
+    // Ensure that offline storage has been initialized on the main thread, as MGLMapView and MGLOfflineStorage do when used directly.
+    // https://github.com/mapbox/mapbox-gl-native-ios/issues/227
+    if ([NSThread.currentThread isMainThread]) {
+        (void)[MGLOfflineStorage sharedOfflineStorage];
+    } else {
         [NSException raise:NSInvalidArgumentException
                     format:@"startWithQueue:completionHandler: must be called from a thread with an active run loop."];
     }

diff --git a/platform/darwin/test/MGLMapSnapshotterTests.m b/platform/darwin/test/MGLMapSnapshotterTests.m
@@ -62,10 +62,6 @@ - (void)setUp {
     [super setUp];
 
     [MGLAccountManager setAccessToken:@"pk.feedcafedeadbeefbadebede"];
-
-    // Initialize file system and run loop before snapshotting.
-    // https://github.com/mapbox/mapbox-gl-native-ios/issues/227
-    (void)[MGLOfflineStorage sharedOfflineStorage];
 }
 
 - (void)tearDown {