563794 - Certain preferences are not syncing across browsers

Status: RESOLVED FIXED

(Firefox :: Sync, defect)

Description

[Tony Chung [:tchung]]

Going from 3.6 -> 3.5 and back to 3.6, there are a few preferences that are not syncing.   I went through the list at: http://hg.mozilla.org/labs/weave/file/1.3b3/source/defaults/preferences/sync.js.in#l34, but it'll be quicker if you can just look and tell if by design or bug.

Prefs List i found:
- Main > Save files to...   location  (3.5 defaults to \Downloads on Mac)
- Security > warn me when sites try to install add-ons 
- Security > warn me when sites try to install add-ons "exceptions" list
- Security > Use a master password checkbox
- Advanced > General: "Check if firefox is default" checkbox
- Content > Fonts: font, size, colors
- Content > Languages: languages
- History > "Remember my browsing history" change # of days

Expected:
- These settings should be sync'd back and forth

Actual:
- the above list did not sync.

Comment 1

[Tony Chung [:tchung]]

This is tested on weave 1.3b3

Comment 2

[Mike Connor [:mconnor]]

(In reply to comment #0)
> Prefs List i found:
> - Main > Save files to...   location  (3.5 defaults to \Downloads on Mac)

By design, this is machine-specific.

> - Security > warn me when sites try to install add-ons 

xpinstall.whitelist.required is not synced, bah.

> - Security > warn me when sites try to install add-ons "exceptions" list

No exception list is synced.  On the roadmap, but not implemented (see also: cookies, popups, images, offline apps)

> - Security > Use a master password checkbox

By design, since we don't sync master password.

> - Advanced > General: "Check if firefox is default" checkbox

By design, per-machine decision.

> - Content > Fonts: font, size, colors

Machine-specific (i.e. netbook vs. 27" iMac :))

> - Content > Languages: languages

Should sync: intl.accept_languages

> - History > "Remember my browsing history" change # of days

Right, we don't sync browser.history_expire_days_min but we do sync browser.history_expire_days.  These prefs are dead on trunk, but that's okay, we should still sync them.

Comment 3

[Mike Connor [:mconnor]]

Net of the above comment, we should sync:

xpinstall.whitelist.required
intl.accept_languages
browser.history_expire_days_min

Comment 4

[Mike Connor [:mconnor]]

Attachment: v1

Comment 5

[Mike Connor [:mconnor]]

http://hg.mozilla.org/labs/weave/rev/dc3b1e7794c9