Comment 46 for bug 119441

Comment on attachment 8339517
proposed fix

Review of attachment 8339517:
-----------------------------------------------------------------

This looks generally reasonable, though currently it is bitrotted so I can't test it out.

Re the test failures, I wonder if you're hitting Linux versus non-Linux line endings or something like that.

::: mail/locales/en-US/chrome/messenger/messengercompose/composeMsgs.properties
@@ +316,5 @@
> ## reply header in composeMsg
> ## user specified
> mailnews.reply_header_originalmessage=-------- Original Message --------
>
> +## Strings used in the corporate reply headers table (mailnews.reply_header_type = 4)

Should we re-use existing strings that we use for forward? I'm not sure there's likely to be a need for separate ones here.

::: mailnews/compose/public/nsIMsgCompose.idl
@@ +247,5 @@
> in nsStringRef aBuf,
> in nsStringRef aSignature,
> in boolean aQuoted,
> + in boolean aHTMLEditor,
> + in boolean aPrefixIsHTML);

Please add documentation for this function, in particular the various parameters and what prefix is used for.