Moving https list check to worker thread

Author: subsymbolic

app/src/main/java/com/duckduckgo/app/browser/BrowserWebViewClient.kt

@@ -21,6 +21,8 @@ import android.graphics.Bitmap
 import android.net.Uri
 import android.net.http.SslError
 import android.os.Build
+import android.support.annotation.AnyThread
+import android.support.annotation.UiThread
 import android.support.annotation.WorkerThread
 import android.webkit.*
 import androidx.core.net.toUri
@@ -32,6 +34,7 @@ import com.duckduckgo.app.statistics.pixels.Pixel.PixelName.HTTPS_UPGRADE_SITE_E
 import com.duckduckgo.app.statistics.pixels.Pixel.PixelParameter
 import timber.log.Timber
 import javax.inject.Inject
+import kotlin.concurrent.thread
 
 
 class BrowserWebViewClient @Inject constructor(
@@ -121,36 +124,45 @@ class BrowserWebViewClient @Inject constructor(
     override fun onReceivedError(view: WebView, errorCode: Int, description: String, failingUrl: String) {
         if (Build.VERSION.SDK_INT < Build.VERSION_CODES.M) {
             val url = failingUrl.toUri()
-            if (isHttpsUpgradeSite(url)) {
-                reportHttpsUpgradeSiteError(url, statusCode = null, error = "WEB_RESOURCE_ERROR_$errorCode")
-            }
+            reportHttpsErrorIfInUpgradeList(url, statusCode = null, error = "WEB_RESOURCE_ERROR_$errorCode")
         }
         super.onReceivedError(view, errorCode, description, failingUrl)
     }
 
     @TargetApi(Build.VERSION_CODES.M)
     override fun onReceivedError(view: WebView, request: WebResourceRequest, error: WebResourceError) {
-        if (request.isForMainFrame && isHttpsUpgradeSite(request.url)) {
-            reportHttpsUpgradeSiteError(request.url, statusCode = null, error = "WEB_RESOURCE_ERROR_${error.errorCode}")
+        if (request.isForMainFrame) {
+            reportHttpsErrorIfInUpgradeList(request.url, statusCode = null, error = "WEB_RESOURCE_ERROR_${error.errorCode}")
         }
         super.onReceivedError(view, request, error)
     }
 
     override fun onReceivedHttpError(view: WebView, request: WebResourceRequest, errorResponse: WebResourceResponse) {
-        if (request.isForMainFrame && isHttpsUpgradeSite(request.url)) {
-            reportHttpsUpgradeSiteError(request.url, errorResponse.statusCode, error = null)
+        if (request.isForMainFrame) {
+            reportHttpsErrorIfInUpgradeList(request.url, errorResponse.statusCode, error = null)
         }
         super.onReceivedHttpError(view, request, errorResponse)
     }
 
+    @UiThread
     override fun onReceivedSslError(view: WebView, handler: SslErrorHandler, error: SslError) {
         val uri = error.url.toUri()
-        if (isHttpsUpgradeSite(uri)) {
-            reportHttpsUpgradeSiteError(uri, null, "SSL_ERROR_${error.primaryError}")
-        }
+        reportHttpsErrorIfInUpgradeList(uri, null, "SSL_ERROR_${error.primaryError}")
         super.onReceivedSslError(view, handler, error)
     }
 
+    @AnyThread
+    private fun reportHttpsErrorIfInUpgradeList(url: Uri, statusCode: Int?, error: String?) {
+
+        if (!url.isHttps) return
+
+        thread {
+            if (httpsUpgrader.isInUpgradeList(url)) {
+                reportHttpsUpgradeSiteError(url, statusCode, error)
+            }
+        }
+    }
+
     private fun reportHttpsUpgradeSiteError(url: Uri, statusCode: Int?, error: String?) {
         val params = mapOf(
             PixelParameter.URL to url.simpleUrl,
@@ -160,10 +172,6 @@ class BrowserWebViewClient @Inject constructor(
         pixel.fire(HTTPS_UPGRADE_SITE_ERROR, params)
     }
 
-    private fun isHttpsUpgradeSite(url: Uri): Boolean {
-        return url.isHttps && httpsUpgrader.isInUpgradeList(url)
-    }
-
     /**
      * Utility to function to execute a function, and then return true
      *