Closed
Bug 235137
Opened 22 years ago
Closed 22 years ago
remove unused checkDownloadDirectory() from pref-navigator.js
Categories
(Firefox :: Settings UI, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: mconnor, Assigned: mconnor)
References
()
Details
Attachments
(1 file)
|
3.99 KB,
patch
|
bugzilla
:
review+
|
Details | Diff | Splinter Review |
this was added in the original fix for bug 178227, which was undone by Ben's new
downloads panel. However, the function was left intact, but isn't used, and its
about 50 lines of cruft.
Also need to remove the unused text strings at
http://lxr.mozilla.org/mozilla/source/browser/components/prefwindow/locale/prefutilities.properties#35
patch today
|
Assignee | |
Comment 1•22 years ago
|
||
|
|
Assignee | |
Comment 2•22 years ago
|
||
Comment on attachment 141908 [details] [diff] [review]
hooray for cruft
ran across this while I was explaining something on IRC, this shouldn't still
be here, nothing uses it
Attachment #141908 -
Flags: review?(firefox)
|
|
||
Comment 3•22 years ago
|
||
Comment on attachment 141908 [details] [diff] [review]
hooray for cruft
Good catch.
Attachment #141908 -
Flags: review?(firefox) → review+
|
|
Assignee | |
Comment 4•22 years ago
|
||
fix checked in 02/24/2004 19:28
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 5•19 years ago
|
||
sorry for bugspam, long-overdue mass reassign of ancient QA contact bugs,
filter on "beltznerLovesGoats" to get rid of this mass change
QA Contact: mconnor → preferences
You need to log in
before you can comment on or make changes to this bug.
Description
•