Add NSURLSession delegation to http_file_source.mm (#16321)

platform/darwin/include/mbgl/interface/native_apple_interface.h

@@ -2,12 +2,16 @@
 
 NS_ASSUME_NONNULL_BEGIN
 
+@class MGLNativeNetworkManager;
+
 @protocol MGLNativeNetworkDelegate <NSObject>
 
 @optional
 
 - (NSString *)skuToken;
 
+- (NSURLSession *)sessionForNetworkManager:(MGLNativeNetworkManager *)networkManager;
+
 @required
 
 - (NSURLSessionConfiguration *)sessionConfiguration;

platform/darwin/src/http_file_source.mm

@@ -250,9 +250,26 @@ std::unique_ptr<AsyncRequest> HTTPFileSource::request(const Resource& resource,
             [MGLNativeNetworkManager.sharedManager startDownloadEvent:url.relativePath type:@"tile"];
         }
 
-        request->task = [impl->session
+        __block NSURLSession *session;
+
+        // Use the delegate's session if there is one, otherwise use the one that
+        // was created when this class was constructed.
+        MGLNativeNetworkManager *networkManager = MGLNativeNetworkManager.sharedManager;
+        if ([networkManager.delegate respondsToSelector:@selector(sessionForNetworkManager:)]) {
+            session = [networkManager.delegate sessionForNetworkManager:networkManager];
+        }
+
+        if (!session) {
+            session = impl->session;
+        }
+
+        assert(session);
+
+        request->task = [session
             dataTaskWithRequest:req
               completionHandler:^(NSData* data, NSURLResponse* res, NSError* error) {
+                session = nil;
+            
                 if (error && [error code] == NSURLErrorCancelled) {
                     [MGLNativeNetworkManager.sharedManager cancelDownloadEventForResponse:res];
                     return;

platform/darwin/src/native_apple_interface.m

@@ -13,16 +13,7 @@ static MGLNativeNetworkManager *instance = nil;
     return instance;
 }
 
-- (NSURLSessionConfiguration *)sessionConfiguration {
++ (NSURLSessionConfiguration *)testSessionConfiguration {
-    if (_delegate && [_delegate respondsToSelector:@selector(sessionConfiguration)]) {
-        return [_delegate sessionConfiguration];
-    }
-    // For testing. Since we get a `nil` return when SDK is modualar, we use this for testing requests.
-    // Same as `[MGLNetworkConfiguration defaultSessionConfiguration]` in `MGLNetworkConfiguration.m`.
-    return [self testSessionConfiguration];
-}
-
-- (NSURLSessionConfiguration *)testSessionConfiguration {
     NSURLSessionConfiguration* sessionConfiguration = [NSURLSessionConfiguration defaultSessionConfiguration];
 
     sessionConfiguration.timeoutIntervalForResource = 30;
@@ -33,41 +24,54 @@ static MGLNativeNetworkManager *instance = nil;
     return sessionConfiguration;
 }
 
+#pragma mark - Optionals
+
 - (NSString *)skuToken {
-    if(_delegate && [_delegate respondsToSelector:@selector(skuToken)]) {
+    if([self.delegate respondsToSelector:@selector(skuToken)]) {
-        return [_delegate skuToken];
+        return [self.delegate skuToken];
     }
     return nil;
 }
 
-- (void)startDownloadEvent:(NSString *)event type:(NSString *)type {
+#pragma mark - Required
-    if (_delegate && [_delegate respondsToSelector:@selector(startDownloadEvent:type:)]) {
+
-        [_delegate startDownloadEvent:event type:type];
+- (NSURLSessionConfiguration *)sessionConfiguration {
+    NSURLSessionConfiguration *configuration = [_delegate sessionConfiguration];
+
+    if (!configuration) {
+        // TODO: Remove
+        NSLog(@"Using testSessionConfiguration");
+
+        // For testing. Since we get a `nil` return when SDK is modular, we use
+        // this for testing requests.
+        // Same as `[MGLNetworkConfiguration defaultSessionConfiguration]` in
+        // `MGLNetworkConfiguration.m`.
+        configuration = [MGLNativeNetworkManager testSessionConfiguration];
     }
+
+    return configuration;
+}
+
+- (void)startDownloadEvent:(NSString *)event type:(NSString *)type {
+    [self.delegate startDownloadEvent:event type:type];
 }
 
 - (void)cancelDownloadEventForResponse:(NSURLResponse *)response {
-    if (_delegate && [_delegate respondsToSelector:@selector(cancelDownloadEventForResponse:)]) {
+    [self.delegate cancelDownloadEventForResponse:response];
-        return [_delegate cancelDownloadEventForResponse:response];
-    }
 }
 
 - (void)stopDownloadEventForResponse:(NSURLResponse *)response {
-    if (_delegate && [_delegate respondsToSelector:@selector(stopDownloadEventForResponse:)]) {
+    [self.delegate stopDownloadEventForResponse:response];
-        return [_delegate stopDownloadEventForResponse:response];
-    }
 }
 
-- (void)debugLog:(NSString *)format, ...{
+- (void)debugLog:(NSString *)format, ... {
-    if (_delegate && [_delegate respondsToSelector:@selector(debugLog:)]) {
+    // TODO: Replace with existing mbgl logging handling.
-        return [_delegate debugLog:format];
+    [self.delegate debugLog:format];
-    }
 }
 
-- (void)errorLog:(NSString *)format, ...{
+- (void)errorLog:(NSString *)format, ... {
-    if (_delegate && [_delegate respondsToSelector:@selector(errorLog:)]) {
+    // TODO: Replace with existing mbgl logging handling.
-        return [_delegate errorLog:format];
+    [self.delegate errorLog:format];
-    }
 }
 
 @end