39254 - Check modes in mimetpla.cpp

Status: RESOLVED WONTFIX

(MailNews Core :: MIME, defect, P3)

Description

[Ben Bucksch (:BenB)]

For bug 31906, trying to understand all the modes
<http://lxr.mozilla.org/seamonkey/source/mailnews/mime/public/nsIMimeStreamConverter.idl#39>
for mime output, some mode handling in mimetpla.cpp look suspicious to me. E.g.
we "skipConversion" for force_user_charset and saveas. As a result, we emitted
bogus HTML for SaveAs HTML ("<>&" not escaped) (changed this with bug 31906).
Not sure, what force_user_charset does (so, I don't touch it), but I think, we
shouldn't skipConversion here either.

I emulate the old mode behaviour in bug 31906, but we should check this
carefully sometime after M16. I need your (Rich's) input for this though, as I
don't understand the modes (their dataflow and usage) well.

Comment 1

[Ben Bucksch (:BenB)]

Not sure what to do with this bug. I fixed several problem with bug 31906, but
there may be more. I'm currently not in the mood to dig into this. Rich, you?

Comment 2

[Ben Bucksch (:BenB)]

> Rich, you?

hm, I doubt it. WONTFIX.