Bug 1070429 - [email/IMAP] premature saving leads to missing bodies

Ugh, thank you for catching that, *so much*.  When I was removing the "i" I thought I had found it was unused.  Clearly I screwed up badly there.  Unfortunately that method doesn't currently have test coverage because it's for ActiveSync 2.5 which our fake-server does not pretend to be.  I can probably make the server provide coverage for that depending on how likely you think the risk of me making another screw-up in that function is :)

My current proposal would be to put the initialization to 0 back in and make the variable name "iHeader".