Created attachment 357648 [details] [diff] [review] The fix I had this patch in my tree as part of the password manager changes but discarded it from the original landing because I forgot it wasn't just tidy up. Toolkit's password manager doesn't support session passwords. Additionally, as master password is now protecting the cache, we should be saving server passwords when the user requests us to. Therefore we can drop the password protect cache preference checking in the SMTP prompt code. I also did some tidy up in these functions - they were unnecessarily complicated, had unneeded checks and could be optimised slightly.
Comment on attachment 357648 [details] [diff] [review] The fix >+ return GetPassword(aUsername); aPassword, surely ;-)
I'll wait for Standard8 to address Neil's comment before applying the patch and running with it.
Created attachment 357701 [details] [diff] [review] The fix v2 Fixed Neil's comment.
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.1b3
Target Milestone: mozilla1.9.1b3 → Thunderbird 3.0b2
You need to log in before you can comment on or make changes to this bug.