Add workaround to dismiss SafariViewController without animation

Author: jdnichollsc

CHANGELOG.md

@@ -30,6 +30,7 @@ in case of vulnerabilities.
 - Activating Open Collective ([#80](https://github.com/proyecto26/react-native-inappbrowser/pull/80)) and Create **CONTRIBUTING.md** to see how to contribute.
 - Added `animated`, `modalPresentationStyle` and `modalTransitionStyle` properties for iOS options ([86f7238](https://github.com/proyecto26/react-native-inappbrowser/commit/86f7238d8eb856b28fae9981ca7bb42b12c43e18)).
 - Present the **SafariViewController** modally or as push instead using the `modalEnabled` property ([4a0d57c](https://github.com/proyecto26/react-native-inappbrowser/commit/4a0d57c73eccaaf45a212853c50aa41520b550c8)).
+- Add workaround to dismiss **SafariViewController** without animation.
 
 ### Removed
 - **com.facebook.infer.annotation** dependency is not required anymore to build for **Android**.

example/App.js

@@ -53,10 +53,10 @@ export default class App extends Component {
           preferredBarTintColor: '#453AA4',
           preferredControlTintColor: 'white',
           readerMode: false,
-          animated: false,
+          animated: true,
           modalPresentationStyle: 'overFullScreen',
           modalTransitionStyle: 'partialCurl',
-          modalEnabled: false,
+          modalEnabled: true,
           // Android Properties
           showTitle: true,
           toolbarColor: '#6200EE',

ios/RNInAppBrowser.m

@@ -138,6 +138,7 @@ - (dispatch_queue_t)methodQueue
   }
 
   UIViewController *ctrl = RCTPresentedViewController();
+
   if (modalEnabled) {
     safariVC.modalPresentationStyle = [self getPresentationStyle: modalPresentationStyle];
     if(animated) {
@@ -166,9 +167,10 @@ - (void)performSynchronouslyOnMainThread:(void (^)(void))block
 - (void)_close
 {
   __weak typeof(self) weakSelf = self;
+  __weak typeof(BOOL) weakAnimated = animated;
   [self performSynchronouslyOnMainThread:^{
     UIViewController *ctrl = RCTPresentedViewController();
-    [ctrl dismissViewControllerAnimated:animated completion:^{
+    [ctrl dismissViewControllerAnimated:weakAnimated completion:^{
       __strong typeof(self) strongSelf = weakSelf;
       if (strongSelf && strongSelf.redirectResolve) {
         strongSelf.redirectResolve(@{
@@ -228,12 +230,13 @@ - (BOOL)initializeWebBrowserWithResolver:(RCTPromiseResolveBlock)resolve andReje
  */
 - (void)safariViewControllerDidFinish:(SFSafariViewController *)controller
 {
-  [controller dismissViewControllerAnimated:animated completion:nil];
+  if (!animated) {
+    [self dismissWithoutAnimation:controller];
+  }
   _redirectResolve(@{
     @"type": @"cancel",
   });
   [self flowDidFinish];
-  [self close];
 }
 
 -(void)flowDidFinish
@@ -277,4 +280,23 @@ - (UIModalTransitionStyle)getTransitionStyle:(NSString *)styleKey {
   return modalTransitionStyle;
 }
 
+- (void)dismissWithoutAnimation:(SFSafariViewController *)controller
+{
+  CATransition* transition = [CATransition animation];
+  transition.duration = 0.0;
+  transition.timingFunction = [CAMediaTimingFunction functionWithName:kCAMediaTimingFunctionEaseInEaseOut];
+  transition.type = kCATransitionFade;
+  transition.subtype = kCATransitionFromBottom;
+
+  controller.view.alpha = 0.05;
+  controller.view.frame = CGRectMake(0.0, 0.0, 0.5, 0.5);
+
+  UIViewController *ctrl = RCTPresentedViewController();
+  NSString* animationKey = @"dismissInAppBrowser";
+  [ctrl.view.layer addAnimation:transition forKey:animationKey];
+  [ctrl dismissViewControllerAnimated:false completion:^{
+    [ctrl.view.layer removeAnimationForKey:animationKey];
+  }];
+}
+
 @end