239902 - cleaning up imap inbox on exit with inbox configured for offline use leaves temp files

Status: RESOLVED FIXED

(MailNews Core :: Backend, defect)

Description

[David :Bienvenu]

If you set up your imap inbox for offline use, and configure the server to
expunge the inbox on exit, we'll leave temp files around because we try to
compact the offline inbox store on exit, but exit before the offline store
compact is finished. This causes us to leave temp files around in the imap
directory. The temporary fix is to not compact the offline store when the msg
window is null, and make it so the msg window is only null when doing a compact
at exit.

Comment 1

[David :Bienvenu]

Attachment: proposed fix

the first part of the fix is to make sure m_window gets set in
nsMsgFolderCompactState so that when we do a compact all, the compact call for
folders after the first call get the msg window passed in. The second part is
to not compact the offline store when we don't have a msg window.

When we re-architect the shutdown code, this will all get cleaned up.

Comment 2

[David :Bienvenu]

Comment on attachment 145631 [details] [diff] [review]
proposed fix

I will try to get this into 1.7

Comment 3

[chris hofmann]

Comment on attachment 145631 [details] [diff] [review]
proposed fix

a=chofmann for 1.6

Comment 4

[chris hofmann]

er, 1.7

Comment 5

[David :Bienvenu]

fixed.