Firefox Sync should synchronize "do not track" pref

VERIFIED FIXED

Status

Cloud Services
General
VERIFIED FIXED
7 years ago
7 years ago

People

(Reporter: dholbert, Assigned: dholbert)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
Just noticed that I'd enabled "do not track" in my Firefox profile on one machine, but that change didn't propagate to the rest of my Sync'ed machines.

This seems like the type of pref that makes a lot of sense to synchronize via sync, since a user who proactively takes the step to request not being tracked will presumably not want to be tracked on *any* of their machines.

(Sid: I assume you agree, but correct me if I'm off base here.)

Can we add this to the list of sync'd prefs?  AIUI, the pref to sync here would be:
  privacy.donottrackheader.enabled
so I think this just involves creating a bool pref named 
  services.sync.prefs.sync.privacy.donottrackheader.enabled
(right?)
> (Sid: I assume you agree, but correct me if I'm off base here.)

Yes, sync it.
(In reply to comment #0)
> Can we add this to the list of sync'd prefs?  AIUI, the pref to sync here would
> be:
>   privacy.donottrackheader.enabled
> so I think this just involves creating a bool pref named 
>   services.sync.prefs.sync.privacy.donottrackheader.enabled
> (right?)

Correct. The services.sync.prefs.sync.* pref would also have to be set to true. It should live in browser/app/profile/firefox.js.
(Assignee)

Comment 3

7 years ago
Created attachment 509498 [details] [diff] [review]
fix

fix as described. I expect it should work -- haven't tested yet, but I'm about to.
Assignee: nobody → dholbert
Status: NEW → ASSIGNED
Attachment #509498 - Flags: review?(philipp)
Attachment #509498 - Flags: review?(philipp) → review+
(Assignee)

Comment 4

7 years ago
Comment on attachment 509498 [details] [diff] [review]
fix

Confirmed that this works in a build w/ this patch.

(successfully synchronized the enabled donottrack pref from profile A to profile B; then I disabled the pref in profile B and resync'd, and confirmed that the pref became disabled in profile A on its next sync.)

Requesting approval to land.
Attachment #509498 - Flags: approval2.0?
(Assignee)

Updated

7 years ago
Depends on: 628197

Updated

7 years ago
Attachment #509498 - Flags: approval2.0? → approval2.0+
Blocks: 631639
(Assignee)

Comment 5

7 years ago
http://hg.mozilla.org/mozilla-central/rev/313e35cc080e
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
verified in beta12 against stage-auth
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.