From 63517a52e4446880735407751cf674f80c263368 Mon Sep 17 00:00:00 2001 From: Christian Sadilek Date: Fri, 13 Mar 2020 11:57:04 -0400 Subject: [PATCH] No issue: Remove unneeded logic to adjust TP --- .../org/mozilla/fenix/AppRequestInterceptor.kt | 15 --------------- 1 file changed, 15 deletions(-) diff --git a/app/src/main/java/org/mozilla/fenix/AppRequestInterceptor.kt b/app/src/main/java/org/mozilla/fenix/AppRequestInterceptor.kt index be4a4b336..047fe8ee7 100644 --- a/app/src/main/java/org/mozilla/fenix/AppRequestInterceptor.kt +++ b/app/src/main/java/org/mozilla/fenix/AppRequestInterceptor.kt @@ -14,7 +14,6 @@ import mozilla.components.concept.engine.request.RequestInterceptor import org.mozilla.fenix.components.metrics.Event import org.mozilla.fenix.ext.components import org.mozilla.fenix.ext.isOnline -import org.mozilla.fenix.ext.settings class AppRequestInterceptor(private val context: Context) : RequestInterceptor { override fun onLoadRequest( @@ -23,7 +22,6 @@ class AppRequestInterceptor(private val context: Context) : RequestInterceptor { hasUserGesture: Boolean, isSameDomain: Boolean ): RequestInterceptor.InterceptionResponse? { - adjustTrackingProtection(context, engineSession) var result: RequestInterceptor.InterceptionResponse? = null // WebChannel-driven authentication does not require a separate redirect interceptor. @@ -41,19 +39,6 @@ class AppRequestInterceptor(private val context: Context) : RequestInterceptor { return result } - private fun adjustTrackingProtection(context: Context, session: EngineSession) { - val trackingProtectionEnabled = context.settings().shouldUseTrackingProtection - if (!trackingProtectionEnabled) { - session.disableTrackingProtection() - } else { - val core = context.components.core - val policy = core.trackingProtectionPolicyFactory - .createTrackingProtectionPolicy(normalMode = true) - core.engine.settings.trackingProtectionPolicy = policy - session.enableTrackingProtection(policy) - } - } - override fun onErrorRequest( session: EngineSession, errorType: ErrorType,