Closed
Bug 709005
Opened 13 years ago
Closed 13 years ago
Sync the browser.sessionstore.restore_on_demand pref
Categories
(Firefox :: General, defect)
Firefox
General
Tracking
()
RESOLVED
FIXED
Firefox 11
People
(Reporter: philikon, Assigned: zpao)
References
Details
(Whiteboard: [fixed-in-fx-team])
Attachments
(1 file, 1 obsolete file)
1.55 KB,
patch
|
rnewman
:
review+
|
Details | Diff | Splinter Review |
The browser.sessionstore.restore_on_demand pref is exposed in the UI in Firefox 8+, but unlike the other exposed session restore related prefs, it's not synced. We should fix this.
Also, for future reference, here's a general rule: if you're introducing a pref that (a) is exposed to the user in the UI, (b) is cross platform, and (c) something you would expect to sync across profiles and/or related to prefs that are already synced: add it to the list of synced prefs!
Assignee | ||
Comment 1•13 years ago
|
||
I can fix [good first bug]s too
Assignee: nobody → paul
Attachment #580487 -
Flags: review?(rnewman)
Assignee | ||
Comment 2•13 years ago
|
||
derp. I can even attach patches that don't have other junk in them.
Attachment #580487 -
Attachment is obsolete: true
Attachment #580487 -
Flags: review?(rnewman)
Attachment #580489 -
Flags: review?(rnewman)
Comment 3•13 years ago
|
||
Comment on attachment 580489 [details] [diff] [review]
Patch v0.2
Review of attachment 580489 [details] [diff] [review]:
-----------------------------------------------------------------
Does what it says on the tin!
Please write up brief steps for QA to verify.
Attachment #580489 -
Flags: review?(rnewman) → review+
Assignee | ||
Comment 4•13 years ago
|
||
Landed in fx-team: https://hg.mozilla.org/integration/fx-team/rev/8e9120a2a121
For QA to verify: check the "don't load tabs until selected" box in the general pref pane. Make sure that gets synced to a second machine.
Whiteboard: [good first bug] → [fixed-in-fx-team]
Comment 5•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 11
You need to log in
before you can comment on or make changes to this bug.
Description
•