STR: 1) In either HTML or Plaintext, create a Filelink 2) Remove the Filelink from the message body 3) Create another Filelink What happens? No Filelink gets inserted into the message body. What's expected? 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] Patch v1
Comment on attachment 612593 [details] [diff] [review] Patch v1 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. -Mike
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