Refactor FXIOS-7586 [v124] Fix deeplink logic (#18539)

* Fix deeplink logic * Remove warning from TODO comment
mozilla-mobile · Feb 7, 2024 · 81fd1c5 · 81fd1c5
1 parent ccbbaf0
commit 81fd1c5
Showing 1 changed file with 16 additions and 1 deletion.
diff --git a/...end/Browser/BrowserViewController/Extensions/BrowserViewController+WebViewDelegates.swift b/...end/Browser/BrowserViewController/Extensions/BrowserViewController+WebViewDelegates.swift
@@ -624,6 +624,21 @@ extension BrowserViewController: WKNavigationDelegate {
             } else {
                 webView.customUserAgent = UserAgent.getUserAgent(domain: url.baseDomain ?? "")
             }
+
+            if navigationAction.navigationType == .linkActivated {
+                if tab.isPrivate {
+                    decisionHandler(.cancel)
+                    webView.load(navigationAction.request)
+                    return
+                } else if UIApplication.shared.canOpenURL(url) {
+                    UIApplication.shared.open(url, options: [.universalLinksOnly: true]) { isAppInstalled in
+                        if isAppInstalled {
+                           // TODO: https://mozilla-hub.atlassian.net/browse/FXIOS-7524
+                        }
+                    }
+                }
+            }
+
             decisionHandler(.allow)
             return
         }
@@ -888,7 +903,7 @@ extension BrowserViewController: WKNavigationDelegate {
         }
 
         // When tab url changes after web content starts loading on the page
-        // We notify the content blocker change so that content blocker status 
+        // We notify the content blocker change so that content blocker status
         // can be correctly shown on beside the URL bar
         tab.contentBlocker?.notifyContentBlockingChanged()
         self.scrollController.resetZoomState()