When doing an LDAP autocomplete, when the user is entering a password, this gives away focus to the nsIAuthPrompt password dialog. This confuses the autocomplete widget into thinking that it's done, and it automatically accepts the currently selected address. If a user isn't paying close attention, they could possibly send mail to someone they didn't intend. Current theory on fixing this: I'll add an attribute to the autocomplete widget which allows the widget to run in a mode where it ignores loss of focus messages until all the sub-sessions have completed. Or perhaps at least until the menu has dropped down. Need to make sure this doesn't add any weird side effects, like leaving the menu up indefinitely if a server doesn't respond.

Discussed in mail news bug meeting. Decided to plus this bug.

First cut at a patch for this. Need to talk some issues over with hewitt before I'm super-confident, however.

Tweaked a few things after discussion with hewitt. Joe, can you review this version?

Fixed bogus comment.

Comment on attachment 87168 [details] [diff] [review] patch, v3 sr=hewitt

Comment on attachment 87168 [details] [diff] [review] patch, v3 r=blake

Fix checked into the trunk; still needs to land on the branch.

yulian, could you verify this on the trunk?

Verified with 20020613 Trunk builds on WIn2K and Linux platforms.

adt1.0.1+ added

Comment on attachment 87168 [details] [diff] [review] patch, v3 Please land this on the 1.0.1 branch. Once there, replace the "mozilla1.0.1+" keyword with the "fixed1.0.1" keyword.

Checked into the branch.

Verified with 20020617 Branch builds on Win2K and Linux platforms.