Merge pull request #4715 from VladiMihaylenko/vm-rel

[ios] SafariViewController
mapsme · Nov 11, 2016 · 3862329 · 3862329
2 parents 4472c50 + 20e11d5
commit 3862329
Showing 1 changed file with 3 additions and 10 deletions.
diff --git a/iphone/Maps/Categories/UIKitCategories.mm b/iphone/Maps/Categories/UIKitCategories.mm
@@ -290,7 +290,7 @@ @implementation UIViewController (SafariDelegateImpl)
 
 - (void)safariViewControllerDidFinish:(SFSafariViewController *)controller
 {
-  [self.navigationController popViewControllerAnimated:YES];
+  [self.navigationController dismissViewControllerAnimated:YES completion:nil];
 }
 
 @end
@@ -299,14 +299,8 @@ @implementation UIViewController (Safari)
 
 - (void)openUrl:(NSURL *)url
 {
-  UIApplication * app = [UIApplication sharedApplication];
-  if ([app canOpenURL:url])
-    [app openURL:url];
-  // TODO(Vlad): Correct implementation of navigation controller's buttons.
-  /*
   NSString * scheme = url.scheme;
-  NSAssert(([scheme isEqualToString:@"http"] || [scheme isEqualToString:@"https"]), @"Incorrect
-  url's scheme!");
+  NSAssert(([scheme isEqualToString:@"http"] || [scheme isEqualToString:@"https"]), @"Incorrect url's scheme!");
   if (isIOS8)
   {
     UIApplication * app = [UIApplication sharedApplication];
@@ -316,8 +310,7 @@ - (void)openUrl:(NSURL *)url
   }
   SFSafariViewController * svc = [[SFSafariViewController alloc] initWithURL:url];
   svc.delegate = self;
-  [self.navigationController pushViewController:svc animated:YES];
-   */
+  [self.navigationController presentViewController:svc animated:YES completion:nil];
 }
 
 @end