Closed Bug 2050750 Opened 2 months ago Closed 2 months ago

Redesigned ETP Custom UI no longer records privacy.ui.fpp.click telemetry for the suspect-fingerprinting controls

Categories

(Firefox :: Settings UI, defect, P3)

defect

Tracking

()

VERIFIED FIXED
154 Branch
Tracking Status
firefox-esr140 --- unaffected
firefox152 --- wontfix
firefox153 --- verified
firefox154 --- verified

People

(Reporter: emz, Assigned: emz)

References

(Regression)

Details

(Keywords: regression)

Attachments

(2 files)

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: nobody → emz
Status: NEW → ASSIGNED

Set release status flags based on info from the regressing bug 2028609

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.

Severity: -- → S3
Priority: -- → P3
Pushed by ezuehlcke@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/7c785f864168 https://hg.mozilla.org/integration/autoland/rev/1cba0ed37886 Record privacy.ui.fpp.click telemetry from the redesigned ETP Custom suspect fingerprinting controls. r=bvandersloot,mstriemer
Regressions: 2052897
Status: ASSIGNED → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → 154 Branch

Given that 153 is our next ESR release, I'm thinking we probably want to get this uplifted to Beta.

Flags: needinfo?(emz)

Request submitted.

Flags: needinfo?(emz)

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.

Flags: needinfo?(ryanvm)

Probably a good idea, yeah.

Flags: needinfo?(ryanvm)

firefox-beta Uplift Approval Request

  • User impact if declined/Reason for urgency: 1. Missing fingerprinting protection settings telemetry (no direct user impact)
  1. Outdated tracker lists right after enabling cryptomining / fingerprinting protections via ETP custom. This will result in a delay until the protection kicks in.
  2. 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
Attachment #9606251 - Flags: approval-mozilla-beta?
Flags: qe-verify+

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

(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.

Attachment #9606251 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
QA Whiteboard: [uplift][qa-ver-needed-c154/b153]

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:

  1. Open about:preferences#etpCustomize
  2. Find Suspected fingerprinters checkbox and flip it off/on
  3. In a new tab open about:glean + F12 / console
  4. 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

Verified as fixed using steps from comment #13 on Windows 11x64 / Mac 15.5 and Firefox build 153.0b11.

Status: RESOLVED → VERIFIED
QA Whiteboard: [uplift][qa-ver-needed-c154/b153] → [uplift][qa-ver-done-c154/b153]
Flags: qe-verify+
QA Contact: mchiorean
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: