1928062 - Update bounce.tracking.protection.enabled_at_startup telemetry to capture all BTP mode states

Status: RESOLVED FIXED

(Core :: Privacy: Anti-Tracking, task, P1)

Comment 1

[Emma Zühlcke [:emz]]

Attachment: Bug 1928062 - Add Glean metric to record BTP mode pref. r=#anti-tracking!

Comment 2

[Emma Zühlcke [:emz]]

Attachment: Bug 1928062 - Update BTP tests to take into account MODE_DISABLED. r=#anti-tracking!

Comment 3

[Emma Zühlcke [:emz]]

Attachment: Bug 1928062 - Add test for BTP mode telemetry. r=#anti-tracking!

Comment 4

[Emma Zühlcke [:emz]]

[Tracking Requested - why for this release]: Looking to uplift this to beta after it has landed in Nightly and the telemetry looks good. This will give us better insight into the Fx133 (train) rollout.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7b9586fbc0f7
Add Glean metric to record BTP mode pref. r=bvandersloot
https://hg.mozilla.org/integration/autoland/rev/43366abe2827
Update BTP tests to take into account MODE_DISABLED. r=anti-tracking-reviewers,bvandersloot
https://hg.mozilla.org/integration/autoland/rev/8a265d369bb1
Add test for BTP mode telemetry. r=anti-tracking-reviewers,bvandersloot

Comment 6

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/7b9586fbc0f7
https://hg.mozilla.org/mozilla-central/rev/43366abe2827
https://hg.mozilla.org/mozilla-central/rev/8a265d369bb1

Comment 7

[Donal Meehan [:dmeehan]]

:pbz please add a beta uplift request on this when you're ready.

Comment 8

[Emma Zühlcke [:emz]]

Comment on attachment 9434468 [details]
Bug 1928062 - Add Glean metric to record BTP mode pref. r=#anti-tracking!

Beta/Release Uplift Approval Request

• User impact if declined/Reason for urgency: We need this updated telemetry as part of the BounceTrackingProtection feature rollout in Fx133
• Is this code covered by automated tests?: Yes
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Low risk patch as it's a telemetry only change. The only slightly risky change is updating the BounceTrackingProtection::GetSingleton call. However this code has good automated test coverage already.
• String changes made/needed:
• Is Android affected?: Yes

For clarification: All 3 patches attached to this bug should be uplifted.

Comment 9

[Donal Meehan [:dmeehan]]

Comment on attachment 9434468 [details]
Bug 1928062 - Add Glean metric to record BTP mode pref. r=#anti-tracking!

Approved for 133.0b6

Comment 10

[Donal Meehan [:dmeehan]]

Comment on attachment 9435093 [details]
Bug 1928062 - Update BTP tests to take into account MODE_DISABLED. r=#anti-tracking!

Approved for 133.0b6

Comment 11

[Donal Meehan [:dmeehan]]

Comment on attachment 9435094 [details]
Bug 1928062 - Add test for BTP mode telemetry. r=#anti-tracking!

Approved for 133.0b6

Comment 12

[Pulsebot]

https://hg.mozilla.org/releases/mozilla-beta/rev/41c268e55f01
https://hg.mozilla.org/releases/mozilla-beta/rev/51313d7745a2
https://hg.mozilla.org/releases/mozilla-beta/rev/5f91eac2ab83