1) In either HTML or Plaintext, create a Filelink
2) Remove the Filelink from the message body
3) Create another Filelink
No Filelink gets inserted into the message body.
If the root/list nodes are not found in the message body, we should re-insert them.
Created attachment 612532 [details] [diff] [review]
WIP Patch v1
Created attachment 612593 [details] [diff] [review]
This patch depends on the patch for bug 742459.
Comment on attachment 612593 [details] [diff] [review]
very nice - my one nit would be naming the method "assertIDsFree" since "assert" in our code generally has a different meaning. Maybe clearNodeIDs? resetNodeIDs?
Created attachment 615364 [details] [diff] [review]
Patch v2 (carrying over r+ from bienvenu)
Thanks! Renamed function to "_resetNodeIDs", and fixed some bitrot.
Committed to comm-central as http://hg.mozilla.org/comm-central/rev/974245c3a39b
Committed to comm-aurora as http://hg.mozilla.org/releases/comm-aurora/rev/9a3c572d0568