101064 - rework ConvertAndLoadComposeWindow() to avoid copying entire body on reply

Status: NEW

(MailNews Core :: Composition, defect)

Description

[(not reading, please use [e-mail] instead)]

fix ConvertAndLoadComposeWindow() to avoid copying entire body on reply

in BuildBodyMessageAndSignature() we do this:

m_compFields->GetBody(&bod); // allocate and copy
nsAutoString bodStr(bod);  // copy again
call ConvertAndLoadComposeWindow() which need access to the buffer because it 
calls TranslateLineEnding()

there's probably more copying going on.

note, there's another caller to ConvertAndLoadComposeWindow(), so keep that in 
mind.

I'm positive we can rework the code to remove at one of the times we copy, 
probably more.

varada, while working on this, I'd suggest stepping through and looking for 
other instances where we copy the body that we can fix.

Comment 1

[scottputterman]

reassigning to sspitzer as part of work on reply perf.