Prefs: Networking cleanup for trunk

RESOLVED WORKSFORME

Status

()

Core
Networking
--
major
RESOLVED WORKSFORME
15 years ago
4 years ago

People

(Reporter: benc, Assigned: benc)

Tracking

Trunk
Future
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(6 obsolete attachments)

(Assignee)

Description

15 years ago
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.
(Assignee)

Updated

15 years ago
Depends on: 203073
(Assignee)

Updated

15 years ago
Depends on: 205044
(Assignee)

Updated

15 years ago
Depends on: 205080
(Assignee)

Updated

15 years ago
Depends on: 205086
(Assignee)

Updated

15 years ago
Depends on: 205097
(Assignee)

Updated

15 years ago
Depends on: 205140
(Assignee)

Updated

15 years ago
Depends on: 205921
(Assignee)

Updated

15 years ago
Depends on: 205146
No longer depends on: 205921
(Assignee)

Comment 1

15 years ago
Created attachment 124710 [details] [diff] [review]
(trunk) patch  /cvsroot/mozilla/modules/libpref/src/init/all.js

has changes for all bugs except bug 205140, which timeless is patching
separately.
(Assignee)

Updated

15 years ago
Attachment #124710 - Flags: review?(gordon)
(Assignee)

Comment 2

15 years ago
Created attachment 124711 [details] [diff] [review]
("browser") patch for /cvsroot/mozilla/browser/app/profile/all.js

removes all prefs except "network.hosts.socks" (removed already) and
"network.http.*.timeout" (removing separately)
(Assignee)

Comment 3

15 years ago
Created attachment 124714 [details] [diff] [review]
("mail") patch /cvsroot/mozilla/mail/app/profile/all.js

http timeout will be handled separately.

Comment 4

15 years ago
Created attachment 128764 [details] [diff] [review]
("calendar") patch for /cvsroot/mozilla/calendar/sunbird/app/profile/all.js

removes all unwanted prefs except "network.http.*.timeout" (removing
separately)

Comment 5

15 years ago
Created attachment 128773 [details] [diff] [review]
("l10n") patch for /cvsroot/mozilla/l10n/langpacks/nscp/ja-JP/defaults/pref/all.js

removes all unwanted prefs except "network.http.*.timeout.
(Assignee)

Updated

15 years ago
Blocks: 33115
(Assignee)

Comment 6

15 years ago
Created attachment 137907 [details] [diff] [review]
trunk patch

leaves network.online, removes network.related.enabled
(Assignee)

Updated

15 years ago
Attachment #124710 - Attachment is obsolete: true
(Assignee)

Comment 7

15 years ago
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)
(Assignee)

Updated

15 years ago
Summary: Prefs: Networking cleanup for 1.4f → Prefs: Networking cleanup for trunk
(Assignee)

Updated

15 years ago
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla1.7alpha

Comment 8

15 years ago
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+
(Assignee)

Comment 9

15 years ago
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 10

15 years ago
Comment on attachment 137907 [details] [diff] [review]
trunk patch

a=mkaply
Attachment #137907 - Flags: approval1.7b+
(Assignee)

Comment 11

15 years ago
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)
(Assignee)

Updated

15 years ago
Depends on: 229211
(Assignee)

Comment 13

15 years ago
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.
(Assignee)

Comment 15

14 years ago
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
(Assignee)

Updated

14 years ago
No longer depends on: 205044
(Assignee)

Comment 16

14 years ago
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
(Assignee)

Comment 17

14 years ago
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 18

14 years ago
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

Comment 19

5 years ago
All child tasks are done.
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.