Until I checked in the patch for 135778, if the prefs code saw a .auth.dn without a .savePassword, it blew away the .auth.dn. This has the side effect that if people switch from newer to older builds and then back, their .auth.dn gets trashed. At first this didn't seem like a big deal, but the more I think about, the more people will just perceive this as "mozilla is flaky". So I'd like to gratuitously write out .savePassword, even though it's not necessary, to avoid this lossage. Somewhere down the road, when everbody has 1.0x or newer, this code can then be thrown out.

This should be an easy fix.

nsbeta1- per mail triage.

One-line patch. Always set .auth.savePassword. We can remove this someday.

Comment on attachment 88404 [details] [diff] [review] patch, v1 sr=bienvenu

Comment on attachment 88404 [details] [diff] [review] patch, v1 r=srilatha

Fix checked into the trunk. Since this bug involves dataloss for people with multiple browsers (eg Netscape and Mozilla) installed (which will be true for most Linux users), I'd like to renominate for landing on the branch. Note that the workaround used here is extremely trivial (4 lines of code).

I concur. Plussing as a beta1+ bug. Yulian, can you try to verify this bug in this mornings builds (looks like Dan checked it in late last night). If we can verify it on the trunk today, we have a chance to get ADT approval before the deadline.

Verified with 20020621 Trunk build on Linux.

adt1.0.1- (on ADT's behalf), sorry ... let's get this one in the next release.

please checkin to the 1.0.1 branch. once there, remove the "mozilla1.0.1+" keyword and add the "fixed1.0.1" keyword.

minusing as there's no one to own/land this change for 1.0.1