From cc9667f72210e16dba785c7e7ca978abdc651719 Mon Sep 17 00:00:00 2001
From: mcarare <mihai.carare.dev@gmail.com>
Date: Thu, 17 Sep 2020 18:21:40 +0300
Subject: [PATCH] For #15150: Do not show open banner if open in app is ON in
 settings.

---
 .../org/mozilla/fenix/browser/OpenInAppOnboardingObserver.kt    | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/app/src/main/java/org/mozilla/fenix/browser/OpenInAppOnboardingObserver.kt b/app/src/main/java/org/mozilla/fenix/browser/OpenInAppOnboardingObserver.kt
index 7b951c9b1..82e6e5aea 100644
--- a/app/src/main/java/org/mozilla/fenix/browser/OpenInAppOnboardingObserver.kt
+++ b/app/src/main/java/org/mozilla/fenix/browser/OpenInAppOnboardingObserver.kt
@@ -36,10 +36,12 @@ class OpenInAppOnboardingObserver(
         }
     }
 
+    @Suppress("ComplexCondition")
     override fun onLoadingStateChanged(session: Session, loading: Boolean) {
         val appLink = appLinksUseCases.appLinkRedirect
 
         if (!loading &&
+            !settings.openLinksInExternalApp &&
             settings.shouldShowOpenInAppBanner &&
             appLink(session.url).hasExternalApp()
         ) {