From 220ddf87ccaaa832687e6505fac30e5bb7fb16bd Mon Sep 17 00:00:00 2001
From: Mugurell <Mugurell@users.noreply.github.com>
Date: Mon, 19 Sep 2022 18:27:40 +0300
Subject: [PATCH] For #12804 - Catch the exception when querying activities

---
 .../feature/app/links/AppLinksUseCases.kt     | 32 ++++++++++++-------
 docs/changelog.md                             |  3 ++
 2 files changed, 23 insertions(+), 12 deletions(-)

diff --git a/components/feature/app-links/src/main/java/mozilla/components/feature/app/links/AppLinksUseCases.kt b/components/feature/app-links/src/main/java/mozilla/components/feature/app/links/AppLinksUseCases.kt
index 88a79ffcd48..4ba1754bd59 100644
--- a/components/feature/app-links/src/main/java/mozilla/components/feature/app/links/AppLinksUseCases.kt
+++ b/components/feature/app-links/src/main/java/mozilla/components/feature/app/links/AppLinksUseCases.kt
@@ -51,11 +51,19 @@ class AppLinksUseCases(
     private val launchInApp: () -> Boolean = { false },
     private val alwaysDeniedSchemes: Set<String> = ALWAYS_DENY_SCHEMES
 ) {
-    @Suppress("QueryPermissionsNeeded") // We expect our browsers to have the QUERY_ALL_PACKAGES permission
+    @Suppress(
+        "QueryPermissionsNeeded", // We expect our browsers to have the QUERY_ALL_PACKAGES permission
+        "TooGenericExceptionCaught",
+    )
     @VisibleForTesting(otherwise = VisibleForTesting.PRIVATE)
     internal fun findActivities(intent: Intent): List<ResolveInfo> {
-        return context.packageManager
-            .queryIntentActivities(intent, PackageManager.GET_RESOLVED_FILTER)
+        return try {
+            context.packageManager
+                .queryIntentActivities(intent, PackageManager.GET_RESOLVED_FILTER)
+        } catch (e: RuntimeException) {
+            Logger("AppLinksUseCases").error("failed to query activities", e)
+            emptyList()
+        }
     }
 
     private fun findDefaultActivity(intent: Intent): ResolveInfo? {
@@ -79,7 +87,7 @@ class AppLinksUseCases(
     inner class GetAppLinkRedirect internal constructor(
         private val includeHttpAppLinks: Boolean = false,
         private val ignoreDefaultBrowser: Boolean = false,
-        private val includeInstallAppFallback: Boolean = false
+        private val includeInstallAppFallback: Boolean = false,
     ) {
         operator fun invoke(url: String): AppLinkRedirect {
             val urlHash = (url + includeHttpAppLinks + ignoreDefaultBrowser + includeHttpAppLinks).hashCode()
@@ -184,7 +192,7 @@ class AppLinksUseCases(
      */
     @Suppress("TooGenericExceptionCaught")
     inner class OpenAppLinkRedirect internal constructor(
-        private val context: Context
+        private val context: Context,
     ) {
         /**
          * Tries to open an external app for the provided [appIntent]. Invokes [failedToLaunchAction]
@@ -197,7 +205,7 @@ class AppLinksUseCases(
         operator fun invoke(
             appIntent: Intent?,
             launchInNewTask: Boolean = true,
-            failedToLaunchAction: () -> Unit = {}
+            failedToLaunchAction: () -> Unit = {},
         ) {
             appIntent?.let {
                 try {
@@ -246,35 +254,35 @@ class AppLinksUseCases(
     val interceptedAppLinkRedirect: GetAppLinkRedirect by lazy {
         GetAppLinkRedirect(
             includeHttpAppLinks = false,
-            includeInstallAppFallback = true
+            includeInstallAppFallback = true,
         )
     }
     val appLinkRedirect: GetAppLinkRedirect by lazy {
         GetAppLinkRedirect(
             includeHttpAppLinks = true,
             ignoreDefaultBrowser = false,
-            includeInstallAppFallback = false
+            includeInstallAppFallback = false,
         )
     }
     val appLinkRedirectIncludeInstall: GetAppLinkRedirect by lazy {
         GetAppLinkRedirect(
             includeHttpAppLinks = true,
             ignoreDefaultBrowser = false,
-            includeInstallAppFallback = true
+            includeInstallAppFallback = true,
         )
     }
     private data class RedirectData(
         val appIntent: Intent? = null,
         val fallbackIntent: Intent? = null,
         val marketplaceIntent: Intent? = null,
-        val resolveInfo: ResolveInfo? = null
+        val resolveInfo: ResolveInfo? = null,
     )
 
     @VisibleForTesting(otherwise = VisibleForTesting.PRIVATE)
     internal data class AppLinkRedirectCache(
         var cacheTimeStamp: Long,
         var cachedUrlHash: Int,
-        var cachedAppLinkRedirect: AppLinkRedirect
+        var cachedAppLinkRedirect: AppLinkRedirect,
     )
 
     companion object {
@@ -284,7 +292,7 @@ class AppLinksUseCases(
         // list of scheme from https://searchfox.org/mozilla-central/source/netwerk/build/components.conf
         internal val ENGINE_SUPPORTED_SCHEMES: Set<String> = setOf(
             "about", "data", "file", "ftp", "http",
-            "https", "moz-extension", "moz-safe-about", "resource", "view-source", "ws", "wss", "blob"
+            "https", "moz-extension", "moz-safe-about", "resource", "view-source", "ws", "wss", "blob",
         )
 
         internal val ALWAYS_DENY_SCHEMES: Set<String> = setOf("jar", "file", "javascript", "data", "about")
diff --git a/docs/changelog.md b/docs/changelog.md
index 372d1135c71..32785959e3b 100644
--- a/docs/changelog.md
+++ b/docs/changelog.md
@@ -11,6 +11,9 @@ permalink: /changelog/
 * [Gecko](https://github.com/mozilla-mobile/android-components/blob/main/buildSrc/src/main/java/Gecko.kt)
 * [Configuration](https://github.com/mozilla-mobile/android-components/blob/main/.config.yml)
 
+* **feature-app-links**
+  * 🚒 Bug fixed [issue #12804](https://github.com/mozilla-mobile/android-components/issues/12804) Speculative fix for a TransactionTooLargeException or RuntimeException when querying activities.
+
 * **all modules**
   * Updated the locally published artifacts to use a single timestamp rather than many [#12902](https://github.com/mozilla-mobile/android-components/issues/12902)