Plain text of the form 'S@Y...where' is linkified incorrectly. I work at Seneca@York (S@Y for short), and this pattern occurs a lot in our email. Email with concurrent dots (e.g., 'a@b..com'), or with a dot trailing the '@' (e.g., 'a@.b') should not be linkified as is being done now.

You need to ask review by setting the r? flag of the attachment to a suitable reviewer... (The CVS log for the file will likely give you someone)

David, thanks for the fix, but scheme-specific tests don't really belong in here (don't mind the basic dot-checking here), the code relies on the generic checks that the nsIURI creation does. Unfortunately, the checks for mailto: are simply absent. Adding them (even far more strict than the patch here) would be highly appreciated. That is bug 32442. Can you fix the bug in that way / code place, please?