No issue: Remove unneeded logic to adjust TP
parent
74433ba27a
commit
63517a52e4
|
@ -14,7 +14,6 @@ import mozilla.components.concept.engine.request.RequestInterceptor
|
||||||
import org.mozilla.fenix.components.metrics.Event
|
import org.mozilla.fenix.components.metrics.Event
|
||||||
import org.mozilla.fenix.ext.components
|
import org.mozilla.fenix.ext.components
|
||||||
import org.mozilla.fenix.ext.isOnline
|
import org.mozilla.fenix.ext.isOnline
|
||||||
import org.mozilla.fenix.ext.settings
|
|
||||||
|
|
||||||
class AppRequestInterceptor(private val context: Context) : RequestInterceptor {
|
class AppRequestInterceptor(private val context: Context) : RequestInterceptor {
|
||||||
override fun onLoadRequest(
|
override fun onLoadRequest(
|
||||||
|
@ -23,7 +22,6 @@ class AppRequestInterceptor(private val context: Context) : RequestInterceptor {
|
||||||
hasUserGesture: Boolean,
|
hasUserGesture: Boolean,
|
||||||
isSameDomain: Boolean
|
isSameDomain: Boolean
|
||||||
): RequestInterceptor.InterceptionResponse? {
|
): RequestInterceptor.InterceptionResponse? {
|
||||||
adjustTrackingProtection(context, engineSession)
|
|
||||||
var result: RequestInterceptor.InterceptionResponse? = null
|
var result: RequestInterceptor.InterceptionResponse? = null
|
||||||
|
|
||||||
// WebChannel-driven authentication does not require a separate redirect interceptor.
|
// WebChannel-driven authentication does not require a separate redirect interceptor.
|
||||||
|
@ -41,19 +39,6 @@ class AppRequestInterceptor(private val context: Context) : RequestInterceptor {
|
||||||
return result
|
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(
|
override fun onErrorRequest(
|
||||||
session: EngineSession,
|
session: EngineSession,
|
||||||
errorType: ErrorType,
|
errorType: ErrorType,
|
||||||
|
|
Loading…
Reference in New Issue