Closed
Bug 311604
Opened 19 years ago
Closed 19 years ago
clearObsoletePrefs is useless
Categories
(Firefox :: General, defect, P3)
Firefox
General
Tracking
()
VERIFIED
FIXED
Firefox 2 alpha1
People
(Reporter: Gavin, Assigned: Gavin)
Details
(Keywords: verified1.8.1)
Attachments
(1 file)
3.54 KB,
patch
|
asaf
:
review+
mconnor
:
approval-branch-1.8.1+
|
Details | Diff | Splinter Review |
It tries to clear saved user prefs that have long been removed from the default pref file, and it does this every time a new window is launched. It should just be removed, since it's no longer useful.
Assignee | ||
Updated•19 years ago
|
Target Milestone: --- → Firefox1.6-
Assignee | ||
Updated•19 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•19 years ago
|
||
timebomb.first_launch_time: not used, gone since forever network.cookie.enable: not used network.cookie.denyRemovedCookies: not used in firefox.js network.cookie.enableForOriginatingWebsiteOnly: not used enableForCurrentSessionOnly: not used, needs to be removed from firefox.js as well network.cookie.warnAboutCookies: still appears to be used, but no default browser.search.defaultengine: Referred to by OpenSearch() in browser.js but nonexistent. (bug 203959 is for cleaning that method) print.use_global_printsettings, print.save_print_settings: Reintroduced in bug 118563, so clearing them is a bug...
Attachment #199760 -
Flags: review?(bugs.mano)
Assignee | ||
Updated•19 years ago
|
Priority: -- → P3
Comment 2•19 years ago
|
||
Comment on attachment 199760 [details] [diff] [review] patch r=mano
Attachment #199760 -
Flags: review?(bugs.mano) → review+
Assignee | ||
Comment 3•19 years ago
|
||
mozilla/browser/base/content/browser.js; new revision: 1.523;
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Updated•19 years ago
|
Flags: blocking-aviary2?
Assignee | ||
Updated•19 years ago
|
Attachment #199760 -
Flags: approval1.8.1?
Updated•19 years ago
|
Flags: blocking-firefox2? → blocking-firefox2-
Updated•19 years ago
|
Attachment #199760 -
Flags: approval1.8.1? → branch-1.8.1?(mconnor)
Updated•19 years ago
|
Attachment #199760 -
Flags: branch-1.8.1?(mconnor) → branch-1.8.1+
Assignee | ||
Comment 4•19 years ago
|
||
Checked in on the branch. mozilla/browser/base/content/browser.js; new revision: 1.479.2.55;
Keywords: fixed1.8.1
Assignee | ||
Updated•18 years ago
|
Target Milestone: Firefox 2 → Firefox 2 alpha1
Assignee | ||
Comment 5•18 years ago
|
||
VERIFIED by code inspection.
Status: RESOLVED → VERIFIED
Keywords: fixed1.8.1 → verified1.8.1
You need to log in
before you can comment on or make changes to this bug.
Description
•