Closed Bug 205077 Opened 22 years ago Closed 11 years ago

Prefs: Networking cleanup for trunk

Categories

(Core :: Networking, defect)

defect
Not set
major

Tracking

()

RESOLVED WORKSFORME
Future

People

(Reporter: benc, Assigned: benc)

References

Details

Attachments

(6 obsolete files)

Over the last two major version releases, we've gotten a much better foundation
for dealing w/ preferences.

Bug 158384 - generic preferences documentation
Bug 198252 - guide to networking prefs.
(http://www.mozilla.org/quality/networking/docs/netprefs.html)

I've also learned enough to use LXR, pull the tree, and write patches to change
the prefs files.

So what I want to do is the following:

1- Document all Networking releate prefs UI.
2- Document all functioning networking prefs.
3- File bugs against any discrepancies.
4- Remove any useless prefs entries from all product trees.

This will be a working meta bug, so process comments should go here, technical
items related to blocking bugs should go in the bug, please.
Depends on: 203073
Depends on: 205044
Depends on: 205080
Depends on: 205086
Depends on: 205097
Depends on: 205140
Depends on: 205921
Depends on: 205146
No longer depends on: 205921
has changes for all bugs except bug 205140, which timeless is patching
separately.
Attachment #124710 - Flags: review?(gordon)
removes all prefs except "network.hosts.socks" (removed already) and
"network.http.*.timeout" (removing separately)
Attached patch ("mail") patch /cvsroot/mozilla/mail/app/profile/all.js (obsolete) (deleted) β€” β€” Splinter Review
http timeout will be handled separately.
removes all unwanted prefs except "network.http.*.timeout" (removing
separately)
removes all unwanted prefs except "network.http.*.timeout.
Blocks: 33115
Attached patch trunk patch (obsolete) (deleted) β€” β€” Splinter Review
leaves network.online, removes network.related.enabled
Attachment #124710 - Attachment is obsolete: true
Comment on attachment 137907 [details] [diff] [review]
trunk patch

review please! Asa says do trunk first, then think about both 1.6f and 1.4.2...
Attachment #137907 - Flags: review?(darin)
Summary: Prefs: Networking cleanup for 1.4f → Prefs: Networking cleanup for trunk
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.7alpha
Comment on attachment 137907 [details] [diff] [review]
trunk patch

r=darin

btw, these "dead" prefs appear in other pref files scattered throughout the
tree.  i recommend searching LXR on the pref names to locate the other files.
Attachment #137907 - Flags: review?(darin) → review+
There was a pref cleanup where other products now do something like copy all.js,
rather than having their own copy in their own tree.

I'll own cleaning up the patch list and hunting down any spurious entries in the
tree.
Comment on attachment 137907 [details] [diff] [review]
trunk patch

a=mkaply
Attachment #137907 - Flags: approval1.7b+
Bryner mentioned a couple more prefs to look at in:

http://bugzilla.mozilla.org/attachment.cgi?id=59999&action=view

I'll start researching those soon.
Comment on attachment 124710 [details] [diff] [review]
(trunk) patch  /cvsroot/mozilla/modules/libpref/src/init/all.js

removing obsolete review request
Attachment #124710 - Flags: review?(gordon)
Depends on: 229211
Comment on attachment 128773 [details] [diff] [review]
("l10n") patch for /cvsroot/mozilla/l10n/langpacks/nscp/ja-JP/defaults/pref/all.js

None of the offending entries appears in this file anymore (per lxr).
Attachment #128773 - Attachment is obsolete: true
The trunk patch has just been checked in by timeless:
2004-04-07 23:43	timeless%mozdev.org 	mozilla/ embedding/ minimo/ all.js 
1.6 	0/9  	Bug 205077 Prefs: Networking cleanup for trunk
patch by benc@chuang.net r=darin a=mkaply
2004-04-07 23:43	timeless%mozdev.org 	mozilla/ modules/ libpref/ src/ init/
all.js 	3.515 	0/10 

please resolve as fixed or leave it if you want to include something from that
in 1.4.2 branch.
Comment on attachment 124711 [details] [diff] [review]
("browser") patch for /cvsroot/mozilla/browser/app/profile/all.js

/mozilla/browser/app/profile/all.js is gone now.
Attachment #124711 - Attachment is obsolete: true
No longer depends on: 205044
Comment on attachment 124714 [details] [diff] [review]
("mail") patch /cvsroot/mozilla/mail/app/profile/all.js

mozilla/mail/app/profile/all.js is gone.
Attachment #124714 - Attachment is obsolete: true
Comment on attachment 128764 [details] [diff] [review]
("calendar") patch for /cvsroot/mozilla/calendar/sunbird/app/profile/all.js

mozilla/calendar/sunbird/app/profile/all.js is gone now.
Attachment #128764 - Attachment is obsolete: true
Comment on attachment 137907 [details] [diff] [review]
trunk patch

mozilla/modules/libpref/src/init/all.js 	3.515
mozilla/embedding/minimo/all.js 	1.6
Attachment #137907 - Attachment is obsolete: true
Target Milestone: mozilla1.7alpha → Future
All child tasks are done.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: