Comment 280 for bug 52667

(From update of attachment 379741)
>+function getServerType(msgHdr)
>+{
>+ try {
>+ return msgHdr.folder.server.type;
>+ } catch (ex) {

catch on a new line

>+ // This empty catch block needs to be here because, although
>+ // msgHdr.folder will be null for .eml files, it will still throw an
>+ // exception when you try to access it.
>+ }
>+ return null;

But please just do this functionality inline instead of as a function. (And i think it should just say that it throws, not that it's null.)

Otherwise looks good to me.