Unverified Commit 9ad318b2 authored by Alex Catarineu's avatar Alex Catarineu Committed by Matthew Finkel
Browse files

Bug 34439: Isolate Icon loader on Android

parent 0cb868ce

components/browser/engine-gecko-beta/src/main/java/mozilla/components/browser/engine/gecko/fetch/GeckoViewFetchClient.kt

+1 −0
Changes for components/browser/engine-gecko-beta/src/main/java/mozilla/components/browser/engine/gecko/fetch/GeckoViewFetchClient.kt: 1 added line, 0 removed lines.
Original line number Diff line number Diff line
@@ -80,6 +80,7 @@ private fun Request.toWebRequest(): WebRequest = WebRequest.Builder(url) 
    .addHeadersFrom(this)

    .addBodyFrom(this)

    .cacheMode(if (useCaches) CACHE_MODE_DEFAULT else CACHE_MODE_RELOAD)

    .origin(origin)

    .build()



private fun WebRequest.Builder.addHeadersFrom(request: Request): WebRequest.Builder {

components/browser/icons/src/main/java/mozilla/components/browser/icons/loader/HttpIconLoader.kt

+2 −1
Changes for components/browser/icons/src/main/java/mozilla/components/browser/icons/loader/HttpIconLoader.kt: 2 added lines, 1 removed line.
Original line number Diff line number Diff line
@@ -51,7 +51,8 @@ class HttpIconLoader( 
            connectTimeout = Pair(CONNECT_TIMEOUT, TimeUnit.SECONDS),

            readTimeout = Pair(READ_TIMEOUT, TimeUnit.SECONDS),

            redirect = Request.Redirect.FOLLOW,

            useCaches = true)

            useCaches = true,

            origin = request.url)



        return try {

            val response = httpClient.fetch(downloadRequest)

components/concept/fetch/src/main/java/mozilla/components/concept/fetch/Request.kt

+2 −1
Changes for components/concept/fetch/src/main/java/mozilla/components/concept/fetch/Request.kt: 2 added lines, 1 removed line.
Original line number Diff line number Diff line
@@ -46,7 +46,8 @@ data class Request( 
    val body: Body? = null,

    val redirect: Redirect = Redirect.FOLLOW,

    val cookiePolicy: CookiePolicy = CookiePolicy.INCLUDE,

    val useCaches: Boolean = true

    val useCaches: Boolean = true,

    val origin: String? = null

) {

    /**

     * A [Body] to be send with the [Request].