631300 - Firefox Sync should synchronize "do not track" pref

Status: VERIFIED FIXED

(Cloud Services :: General, defect)

Description

[Daniel Holbert [:dholbert]]

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?)

Comment 1

[Sid Stamm [:geekboy or :sstamm]]

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

Yes, sync it.

Comment 2

[Philipp von Weitershausen [:philikon]]

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

Comment 3

[Daniel Holbert [:dholbert]]

Attachment: fix

fix as described. I expect it should work -- haven't tested yet, but I'm about to.

Comment 4

[Daniel Holbert [:dholbert]]

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.

Comment 5

[Daniel Holbert [:dholbert]]

http://hg.mozilla.org/mozilla-central/rev/313e35cc080e

Comment 6

[Tracy Walker [:tracy]]

verified in beta12 against stage-auth