Created attachment 315401 [details] [diff] [review] The fix Thunderbird leaks an nsLocaleFile (and conceivably its nsStringBuffer), probably due to bug 388833. According to my standalone-tinderbox I have been running we get these leaks on the Mail RLk test: TOTAL 384 0.00% nsLocalFile 360 0.00% nsStringBuffer 24 0.00% I picked the fix up after doing a bit of research and finding bug 389779 - basically we need to add a bit of code to nsMailApp.cpp to do things cleanly. See attached patch, with this patch in, and the current form of the tinderbox tests, Mail RLk goes down to 0B for TB. This probably accounts for some, but not all of the leaks in bug 426963 (yes we probably need to expand the tinderbox tests a bit more).
Yeah, it's a fictional leak, but since clean leak logs are good the extra loginit/logterm pair will get you where you want to be. r=me, FWIW, though a mailnews peer should still give MOA.
Comment on attachment 315401 [details] [diff] [review] The fix Moving review as dmose hasn't got privs in mail/ yet
Comment on attachment 315401 [details] [diff] [review] The fix After careful study and long analysis, I believe what bsmedberg tells me ;) r=philringnalda
Patch checked in -> fixed.