729212 - Port |Bug 717225 - Figure out why folderTree.json is getting corrupted, and improve stability during writing| to SeaMonkey

Status: RESOLVED FIXED

(SeaMonkey :: MailNews: Address Book & Contacts, defect, P3)

Comment 1

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) abTrees.js: Use a safe-file-output-stream [Checked in: See comments 4 and 5]

Comment 2

[Serge Gautherie (:sgautherie)]

Ping for review.

Comment 3

[Karsten Düsterloh]

Comment on attachment 599268 [details] [diff] [review]
(Av1) abTrees.js: Use a safe-file-output-stream
[Checked in: See comments 4 and 5]

>+      let foStream = Components.classes["@mozilla.org/network/safe-file-output-stream;1"]
>                                .createInstance(Components.interfaces.nsIFileOutputStream);
>-
>       foStream.init(file, 0x02 | 0x08 | 0x20, 0666, 0);
>       foStream.write(data, data.length);
>+      foStream.QueryInterface(Ci.nsISafeOutputStream).finish();

You need to replace the Ci with Components.interfaces here as well.

r/moa=me with that.

Sorry for the long delay.

Comment 4

[Serge Gautherie (:sgautherie)]

Comment on attachment 599268 [details] [diff] [review]
(Av1) abTrees.js: Use a safe-file-output-stream
[Checked in: See comments 4 and 5]

http://hg.mozilla.org/comm-central/rev/329a4f21d574

Comment 5

[Jens Hatlak (:InvisibleSmiley)]

Comment on attachment 599268 [details] [diff] [review]
(Av1) abTrees.js: Use a safe-file-output-stream
[Checked in: See comments 4 and 5]

http://hg.mozilla.org/releases/comm-aurora/rev/7b7fa546c0c3