Closed
Bug 289640
Opened 19 years ago
Closed 19 years ago
Remove unused preference security.ui.enable
Categories
(Core :: Networking, defect)
Core
Networking
Tracking
()
VERIFIED
FIXED
mozilla1.9alpha1
People
(Reporter: djcater+bugzilla, Assigned: djcater+bugzilla)
References
()
Details
Attachments
(1 file)
850 bytes,
patch
|
darin.moz
:
review+
KaiE
:
superreview+
|
Details | Diff | Splinter Review |
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8b2) Gecko/20050408 Firefox/1.0+ security.ui.enable is NEVER used in any code anywhere. Here is the only place where it appears: http://lxr.mozilla.org./mozilla/source/netwerk/base/public/security-prefs.js#47 And here is the CVS Log for that checkin: http://bonsai.mozilla.org./cvslog.cgi?file=mozilla/netwerk/base/public/security-prefs.js#1.4 And here is what was checked in: http://bonsai.mozilla.org./cvsview2.cgi?diff_mode=context&whitespace_mode=show&file=security-prefs.js&branch=&root=/cvsroot&subdir=mozilla/netwerk/base/public&command=DIFF_FRAMESET&rev1=1.3&rev2=1.4 Reproducible: Always Steps to Reproduce: 1. View the above 3 URIs, and also about:config 2. Search LXR for where the preference is used. Actual Results: about:config shows the preference (defaults to true.) LXR shows nowhere that uses the preference. Expected Results: Either: a) Preference does something. Or b) Prefence doesn't exist.
Severity: normal → trivial
Status: UNCONFIRMED → NEW
Ever confirmed: true
Doug, is there any need for this preference you checked in years ago?
Assignee: bugs → DJCater
Status: NEW → ASSIGNED
Attachment #207893 -
Flags: superreview?(darin)
Attachment #207893 -
Flags: review?(dougt)
Component: Preferences → Networking
Flags: review?(dougt)
OS: Linux → All
Product: Firefox → Core
Hardware: PC → All
Attachment #207893 -
Flags: review?(dougt)
Comment 2•19 years ago
|
||
Comment on attachment 207893 [details] [diff] [review] Remove the preference i do not think it is needed.
Attachment #207893 -
Flags: superreview?(kengert)
Attachment #207893 -
Flags: superreview?(darin)
Attachment #207893 -
Flags: review?(dougt)
Attachment #207893 -
Flags: review?(darin)
Updated•19 years ago
|
Attachment #207893 -
Flags: superreview?(kengert) → superreview+
Updated•19 years ago
|
Attachment #207893 -
Flags: review?(darin) → review+
Comment 4•19 years ago
|
||
mozilla/netwerk/base/public/security-prefs.js; new revision: 1.13;
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9alpha
Comment 5•19 years ago
|
||
Is this also in branch? If it is indeed trivial, is the impact also trivial? I know streamlining both code and code size are always important around here. Thanks.
You need to log in
before you can comment on or make changes to this bug.
Description
•