1791947 - Remove nsIMsgParseMailMsgState.GetAllHeaders() in favor of nsIMsgParseMailMsgState.headers

Status: NEW

(MailNews Core :: Backend, task)

Description

[Ben Campbell]

Remove nsIMsgParseMailMsgState.GetAllHeaders().
The only two uses are in nsImapMailFolder.cpp and nsParseMailbox.cpp.
These should use nsIMsgParseMailMsgState.headers attribute instead.

Reasoning:
.headers provides more-or-less the raw header block of a message.
.GetAllHeaders() provides the same, but with a NUL char replacing the EOL '\r\n'.

Ostensibly, this is because the filter matching requires a NUL-terminated list, but in fact the code that does the matching (nsMsgBodyHandler) handles doing it's own '\r\n' parsing just fine, and actually has to do some considerable hoop-jumping in order to handle NUL-terminated headers!