Commit f91ab1cb authored by brizental's avatar brizental Committed by Pier Angelo Vendrame
Browse files

fixup! [android] Disable features and functionality

Bug 45145: Disable app links feature

This feature starts an observer to watch for potential redirects
to other apps. This commit prevents that observer from ever starting.
parent 6e0f6a23
Loading
Loading
Loading
Loading
+0 −27
Changes for mobile/android/fenix/app/src/main/java/org/mozilla/fenix/browser/BaseBrowserFragment.kt: 0 added lines, 27 removed lines.
Original line number Diff line number Diff line
@@ -431,33 +431,6 @@ abstract class BaseBrowserFragment :

        initializeUI(view)

        appLinksFeature.set(
            feature = AppLinksFeature(
                context = requireContext(),
                store = requireComponents.core.store,
                fragmentManager = parentFragmentManager,
                sessionId = customTabSessionId,
                dialog = appLinksPromptDialog(),
                launchInApp = { requireComponents.settings.shouldOpenLinksInApp(customTabSessionId != null) },
                loadUrlUseCase = requireComponents.useCases.sessionUseCases.loadUrl,
                shouldPrompt = { requireComponents.settings.shouldPromptOpenLinksInApp() },
                alwaysOpenCheckboxAction = {
                    requireComponents.settings.openLinksInExternalApp =
                        requireContext().getString(R.string.pref_key_open_links_in_apps_always)
                },
                failedToLaunchAction = { fallbackUrl ->
                    fallbackUrl?.let {
                        val appLinksUseCases = requireComponents.useCases.appLinksUseCases
                        val getRedirect = appLinksUseCases.appLinkRedirect
                        val redirect = getRedirect.invoke(fallbackUrl)
                        appLinksUseCases.openAppLink.invoke(redirect.appIntent)
                    }
                },
            ),
            owner = this,
            view = binding.root,
        )

        if (customTabSessionId == null) {
            // We currently only need this observer to navigate to home
            // in case all tabs have been removed on startup. No need to