Closed Bug 612816 Opened 9 years ago Closed 9 years ago

Remove some dead prefs from firefox.js

Categories

(Firefox :: Preferences, defect)

defect
Not set

Tracking

()

VERIFIED FIXED
Firefox 5

People

(Reporter: philor, Assigned: philor)

Details

Attachments

(1 file)

Attached patch rmSplinter Review
general.startup.browser is used by suite/browser/nsBrowserContentHandler.js to determine which SeaMonkey windows to open on startup. Prior to bug 276588, xpfe's nsAppStartup::CreateStartupState would read the general.startup pref branch, and open whatever said it wanted to be opened, but we haven't actually used it since 2005 (other than the way browser/base/content/test/browser_utilityOverlay.js uses it to test getting a bool pref, but browser.search.openintab will do for a stable bool pref that testing profiles won't have any reason to change)

profile.allow_automigration was used by profile/src/nsProfile.cpp to determine whether to automatically migrate a Netscape 4.x profile, but that got the axe after 1.8

custtoolbar.personal_toolbar_folder was used by the xpfe bookmarks service, as one of three different ways to decide which bookmarks folder was the ptf, and nobody should ever have to be reminded about the xpfe bookmarks service
Attachment #491105 - Flags: review?(gavin.sharp)
Attachment #491105 - Flags: review?(gavin.sharp) → review+
Attachment #491105 - Flags: approval2.0?
Whiteboard: [needs approval]
Attachment #491105 - Flags: approval2.0?
Whiteboard: [needs approval]
Whiteboard: [needs a trunk]
http://hg.mozilla.org/mozilla-central/rev/3f34857aafdb
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Whiteboard: [needs a trunk]
Target Milestone: --- → Firefox4.2
Verified on Mozilla/5.0 (Windows NT 6.1; rv:2.2a1pre) Gecko/20110403 Firefox/4.2a1pre

The following prefs have been removed:

general.startup.browser
profile.allow_automigration
custtoolbar.personal_toolbar_folder
Status: RESOLVED → VERIFIED
Target Milestone: Firefox5 → Firefox 5
I have seen discussions that profile.allow_automigration also controls importing IE settings into FF. Is that no longer the case?
And what about profile.confirm_automigration ?
You need to log in before you can comment on or make changes to this bug.