Redesigned ETP Custom UI no longer records privacy.ui.fpp.click telemetry for the suspect-fingerprinting controls
Categories
(Firefox :: Settings UI, defect, P3)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox-esr140 | --- | unaffected |
| firefox152 | --- | wontfix |
| firefox153 | --- | verified |
| firefox154 | --- | verified |
People
(Reporter: emz, Assigned: emz)
References
(Regression)
Details
(Keywords: regression)
Attachments
(2 files)
|
48 bytes,
text/x-phabricator-request
|
Details | Review | |
|
48 bytes,
text/x-phabricator-request
|
phab-bot
:
approval-mozilla-beta+
|
Details | Review |
With the settings redesign enabled (browser.settings-redesign.enabled), interacting with the Suspect Fingerprinting checkbox and scope menu in the ETP Custom UI no longer records the privacy.ui.fpp.click Glean telemetry.
In the legacy UI, browser/components/preferences/privacy.js records Glean events on these controls: Glean.privacyUiFpp.click.checkbox.record({ checked }) on the fingerprinting protection checkbox and Glean.privacyUiFpp.click.menu.record({ value }) on the menulist (wired in initContentBlocking()).
In the redesign (browser/components/preferences/config/privacy.mjs), the etpCustomSuspectFingerprintingProtectionEnabled and etpCustomSuspectFingerprintingProtectionEnabledContext settings flip privacy.fingerprintingProtection / privacy.fingerprintingProtection.pbmode but have no onUserChange handler and record no telemetry. The probe (privacy.ui.fpp.click, defined in browser/components/preferences/metrics.yaml, expires: never) is therefore no longer collected from the redesigned UI.
Fix: add onUserChange handlers to these settings recording Glean.privacyUiFpp.click.checkbox / .menu to match legacy. May warrant a data-review ping since it restores collection from the new UI path.
Found while auditing legacy-vs-redesign ETP Custom toggle behavior for bug 2050000.
| Assignee | ||
Updated•2 months ago
|
Comment 1•2 months ago
|
||
Set release status flags based on info from the regressing bug 2028609
| Assignee | ||
Comment 2•2 months ago
|
||
The legacy ETP Custom UI records the privacy.ui.fpp.click Glean events when
the user interacts with the suspect fingerprinting checkbox and scope menu.
The settings redesign reimplemented these controls but recorded no telemetry,
so the probe (which never expires) stopped being collected from the new UI.
Record Glean.privacyUiFppClick.checkbox / .menu from the onUserChange handlers
of the etpCustomSuspectFingerprintingProtectionEnabled and
etpCustomSuspectFingerprintingProtectionEnabledContext settings, mirroring the
old design's instrumentation.
Updated•2 months ago
|
Comment 4•2 months ago
|
||
| bugherder | ||
Comment 5•2 months ago
|
||
Given that 153 is our next ESR release, I'm thinking we probably want to get this uplifted to Beta.
| Assignee | ||
Comment 7•2 months ago
|
||
Hmm in that case would you also like uplifts for the other ETP custom settings bugs? That's Bug 2050000 and Bug 2050749. Given their severity I had not considered them for an uplift prior.
Comment 9•2 months ago
|
||
firefox-beta Uplift Approval Request
- User impact if declined/Reason for urgency: 1. Missing fingerprinting protection settings telemetry (no direct user impact)
- Outdated tracker lists right after enabling cryptomining / fingerprinting protections via ETP custom. This will result in a delay until the protection kicks in.
- Social media tracker blocking feature not controllable via ETP custom.
- Code covered by automated testing?: yes
- Fix verified in Nightly?: yes
- Needs manual QE testing?: yes
- Steps to reproduce for manual QE testing: Bug 2050000: See bug description (comment 0)
Bug 2050749, Bug 2050749 are sufficiently covered by automated tests and don't need manual testing.
- Risk associated with taking this patch: low
- Explanation of risk level: Fairly small code changes only affecting the ETP custom subpanel. Good automated test coverage.
- String changes made/needed?: no
- Is Android affected?: no
| Assignee | ||
Comment 10•2 months ago
|
||
The legacy ETP Custom UI records the privacy.ui.fpp.click Glean events when
the user interacts with the suspect fingerprinting checkbox and scope menu.
The settings redesign reimplemented these controls but recorded no telemetry,
so the probe (which never expires) stopped being collected from the new UI.
Record Glean.privacyUiFppClick.checkbox / .menu from the onUserChange handlers
of the etpCustomSuspectFingerprintingProtectionEnabled and
etpCustomSuspectFingerprintingProtectionEnabledContext settings, mirroring the
old design's instrumentation.
Original Revision: https://phabricator.services.mozilla.com/D308995
| Assignee | ||
Comment 11•2 months ago
|
||
(In reply to Ryan VanderMeulen [:RyanVM] from comment #8)
Probably a good idea, yeah.
Ok I've requested an uplift covering the other bugs. The patches landed as a stack in Nightly so uplifting them together is easiest. The base patch has already been uplifted.
Updated•2 months ago
|
Updated•2 months ago
|
Comment 12•2 months ago
|
||
| uplift | ||
Updated•2 months ago
|
Comment 13•2 months ago
|
||
Reproduced the issue of glean telemetry not being recorded for Suspected fingerprinters with affected Nightly 154.0a1 (2026-07-04) then verified it fixed with Nightly 154.0a1 (2026-07-10) on Windows 11, Mac 13 and Ubuntu 24.
STR:
- Open
about:preferences#etpCustomize - Find
Suspected fingerprinterscheckbox and flip it off/on - In a new tab open about:glean + F12 / console
- In the console input
Glean.privacyUiFppClick.checkbox.testGetValue()
ER:
There should be two arrays recorded, on for the checkbox false and one for the checkbox true
Comment 14•2 months ago
|
||
Verified as fixed using steps from comment #13 on Windows 11x64 / Mac 15.5 and Firefox build 153.0b11.
Description
•