19177 - user_pref("string:network.proxy.http", "proxyhost") fails

Status: VERIFIED FIXED

(SeaMonkey :: Preferences, defect, P3)

Description

[tomh]

The default prefs.js file is created with string: labels on the
proxy lines, and the proxy port gets entered with quotes around it.
It doesn't work. In GetIntPref, the "network.proxy.http" name is
marked as 'no user value' and the default gets returned. Same for
the port. Changing the prefs.js file to read:
user_pref("network.proxy.http", "proxyhost");
user_pref("network.proxy.http_port", port); where port is numeric (no quotes)
works. So the bug is either
the fact that the prefs.js file is created with string: labels, or
that libPref doesn't handle them.

Comment 1

[Paul MacQuiddy]

setting tever has qa contact for proxy bug

Comment 2

[Sean Richardson]

Based on issues, this is a DUP of bug 16274 "Writing out bad proxy prefs" ...
both bugs are ASSIGNED and M13

Is this a "libPref" problem or a "Networking" problem? Guessing "libPref",
so that would make bug 16274 a DUPLICATE of this bug.

According to [e-mail] in bug 16274,
>Both the string: before the pref name and the quotes around the port number
>are bogus. Changing these manually fixed the problem.

Comment 3

[neeti]

Looks like this is case of writing bad proxy prefs from Necko. Marking it a
duplicate of 16274.

*** This bug has been marked as a duplicate of 16274 ***

Comment 4

[Gagan]

Necko doesn't write out any prefs. This is a case of bad prefs being written out

from the preference dialog box. I am correcting this by setting the dup back to

the prefs owner. The other bug is a dup of this one.

Comment 5

[Gagan]

*** Bug 16274 has been marked as a duplicate of this bug. ***

Comment 6

[neeti]

Reassigning this to Matt. Looks like a Pref UI bug.

Comment 7

[sairuh (rarely reading bugmail)]

spam: in my testing realm, so reassigning qa contact to me, en masse.

Comment 8

[matt]

I believe this was fixed a while ago.
This is what my pref wrote out:
user_pref("network.proxy.http", "what");
user_pref("network.proxy.http_port", 99);

Comment 9

[sairuh (rarely reading bugmail)]

my bad: this should prolly go back to tever for verif...

Comment 10

[leger]

Bulk move of all Pref UI component bugs to new Preferences component.  Pref UI 
component will be deleted.

Comment 11

[Tom Everingham]

verif.
Linux 2000071308