Closed Bug 1513076 Opened 10 months ago Closed 10 months ago

Do Not Track string no longer makes sense in Firefox 65

Categories

(Firefox :: Protections UI, defect, P1)

65 Branch
defect

Tracking

()

VERIFIED FIXED
Firefox 66
Tracking Status
firefox64 --- unaffected
firefox65 + verified
firefox66 + verified

People

(Reporter: tanvi, Assigned: ewright)

References

(Blocks 1 open bug)

Details

(Whiteboard: [privacy65])

Attachments

(1 file)

In the section for Do Not Track in about:preferences#privacy, it says:
Send websites a “Do Not Track” signal that you don’t want to be tracked.
* Always
* Only when Nightly is set to block Detected Trackers

The second option no longer makes sense, because we don't use the terminology "Detected Trackers" anymore.  We need to change this string.

Betsy has proposed:
Only when Nightly is set to block known trackers
Assignee: nobody → ewright
Status: NEW → ASSIGNED
Pushed by ewright@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/663a91125ddf
Do Not Track string change for Firefox 65. r=flod
Comment on attachment 9030330 [details]
Bug 1513076 - Do Not Track string change for Firefox 65.

[Beta/Release Uplift Approval Request]

Feature/Bug causing the regression: Bug 1473978

User impact if declined: This is a simple string change, if declined we will have misleading terminology in about:preferences.

Is this code covered by automated tests?: Yes

Has the fix been verified in Nightly?: No

Needs manual test from QE?: Yes

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): the only risk is that this is a string change, so may not be fully translated in time.

String changes made/needed: yes, 1 change
Attachment #9030330 - Flags: approval-mozilla-beta?
EDIT:

> Is this code covered by automated tests?: no
> Needs manual test from QE?: no
(In reply to Erica Wright [:ewright] from comment #3)
> String changes made/needed: yes, 1 change

I'm fine with uplifting this.
(In reply to Francesco Lodolo [:flod] from comment #5)
> (In reply to Erica Wright [:ewright] from comment #3)
> > String changes made/needed: yes, 1 change
> 
> I'm fine with uplifting this.

Thank you Flod.
Target Milestone: Firefox 65 → ---
Comment on attachment 9030330 [details]
Bug 1513076 - Do Not Track string change for Firefox 65.

[Triage Comment]
Fixes a misleading string in Firefox 65. Has Flod's blessing for the string change. Approved for 65.0b4.
Attachment #9030330 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Flags: qe-verify+
https://hg.mozilla.org/mozilla-central/rev/663a91125ddf
Status: ASSIGNED → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 66
I was able to reproduce this bug using an affected Nightly build from 2018-12-10.

This is verified as fixed on latest Nightly 66.0a1 (2018-12-11) and Beta 65.0b4 (20181211223337) on the following OSes: Win 10 x64, Mac OS X 10.11 and Ubuntu 16.04 x64. I can confirm that the second option is now called "Only when Firefox/Nightly is set to block known trackers" in about:preferences#privacy.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.