220529 - Remove obsolete wallet prefs from firefox.js

Status: RESOLVED DUPLICATE of bug 224578

(Toolkit :: Password Manager, defect)

Description

[Richey]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.5) Gecko/20030925
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.5) Gecko/20030925

If I define an alternate location for the password file, it is ignored by
Mozilla (and Firebird, by the way). Entered paths (example:
user_pref("signon.SignonFileName", "C:\\Data\\reg\\61256111.s"); ) are ignored
completely.

Reproducible: Always

Steps to Reproduce:
1. add an alternate path for signon.txt or the Mozilla password file in user.js
2. run Mozilla or Firebird
3. enter a web page of which a password was already stored

Actual Results:  
no password is automatically entered (or saved if you agree to save it after
re-entering it). The setting is ignored completely; newly entered passwords go
to "nowhere".

Expected Results:  
evaluate the user.js setting (also if it contains a path) and work as intended
(enter saved passwords and save them to the configured password file)

Comment 1

[Bill Mason]

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

Comment 2

[Bill Mason]

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

Comment 3

[Bill Mason]

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

Comment 4

[Bill Mason]

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

Comment 5

[Mike Connor [:mconnor]]

signon.txt isn't used by Firebird's password manager (its a random filename
ending in .s in your profile).  As far as I can make out, wallet used this pref,
and since we don't use wallet, it's irrelevant.

It should get whacked out of all.js so as not to confuse people though, since it
no longer has any effect.  Morphing bug to that effect.

-> taking

Comment 6

[Mike Connor [:mconnor]]

bryner, am I correct in assuming that any wallet prefs are obsolete now?

Comment 7

[David P James]

Come on Mike - you know you shouldn't be both assignee and QA for a bug :)

Comment 8

[Mike Connor [:mconnor]]

does that mean I can reassign you as QA for all the bugs I'm doubled on? :)

Comment 9

[Robert Parenton]

Tweaking summary since we use firefox.js instead of all.js now.

Comment 10

[Steffen Wilberg]

What exactly do you want to remove here? The wallet.* prefs are gone. The signon
prefs are in use: signon.SignonFileName points to the passwords file
(signons.txt currently); signon.rememberSignons is the bool pref you set in
options->privacy->saved passwords->remember passwords. I haven't tested
signon.expireMasterPassword though.

Comment 11

[Mike Connor [:mconnor]]

I'm not sure this is valid, but the prefs don't seem to show up in any of the
new password manager code, I may be wrong on how they're added (i.e. signon. is
appended somehow to strings, or something).

-> default owner/QA

Comment 12

[Simon Paquet [:sipaq]]

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