Skip to content

Commit db9c2cb

Browse files
MozLandoAmejia481
andcommitted
4430: Closes mozilla-mobile#4428: Remove fingerprint from the tracking protection recommended policy. r=pocmo a=Amejia481 Co-authored-by: Arturo Mejia <[email protected]>
2 parents e608129 + a8825bd commit db9c2cb

File tree

2 files changed

+4
-5
lines changed

2 files changed

+4
-5
lines changed

components/concept/engine/src/main/java/mozilla/components/concept/engine/EngineSession.kt

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -220,12 +220,13 @@ abstract class EngineSession(
220220
SCRIPTS_AND_SUB_RESOURCES(1 shl 9999),
221221

222222
RECOMMENDED(AD.id + ANALYTICS.id + SOCIAL.id + TEST.id + MOZILLA_SOCIAL.id +
223-
CRYPTOMINING.id + FINGERPRINTING.id),
223+
CRYPTOMINING.id),
224224

225225
/**
226-
* Combining the [RECOMMENDED] categories plus [CRYPTOMINING] and [FINGERPRINTING].
226+
* Combining the [RECOMMENDED] categories plus [SCRIPTS_AND_SUB_RESOURCES]
227+
* and [FINGERPRINTING].
227228
*/
228-
STRICT(RECOMMENDED.id + SCRIPTS_AND_SUB_RESOURCES.id)
229+
STRICT(RECOMMENDED.id + SCRIPTS_AND_SUB_RESOURCES.id + FINGERPRINTING.id)
229230
}
230231

231232
companion object {

components/concept/engine/src/test/java/mozilla/components/concept/engine/EngineSessionTest.kt

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -688,7 +688,6 @@ class EngineSessionTest {
688688
assertTrue(recommendedPolicy.contains(TrackingCategory.SOCIAL))
689689
assertTrue(recommendedPolicy.contains(TrackingCategory.TEST))
690690

691-
assertTrue(recommendedPolicy.contains(TrackingCategory.FINGERPRINTING))
692691
assertTrue(recommendedPolicy.contains(TrackingCategory.CRYPTOMINING))
693692
assertFalse(recommendedPolicy.contains(TrackingCategory.CONTENT))
694693

@@ -699,7 +698,6 @@ class EngineSessionTest {
699698
assertTrue(strictPolicy.contains(TrackingCategory.ANALYTICS))
700699
assertTrue(strictPolicy.contains(TrackingCategory.SOCIAL))
701700
assertTrue(strictPolicy.contains(TrackingCategory.TEST))
702-
assertTrue(strictPolicy.contains(TrackingCategory.FINGERPRINTING))
703701
assertTrue(strictPolicy.contains(TrackingCategory.CRYPTOMINING))
704702
assertFalse(strictPolicy.contains(TrackingCategory.CONTENT))
705703
}

0 commit comments

Comments
 (0)