Created attachment 8515603 [details] [diff] [review] WIP v1 We can put more important information on the download manager view. For example, sender of attachment file is one of the important information. Attaching patch shows sender info in the view.
Created attachment 8517977 [details] [diff] [review] WIP v2 Implemented MsgDownloads.saveAllAttachments.
Attachment #8515603 - Attachment is obsolete: true
Created attachment 8518084 [details] [diff] [review] WIP v3 Used also Downloads.jsm on detaching.
Attachment #8517977 - Attachment is obsolete: true
Created attachment 8518109 [details] [diff] [review] WIP v4 Added received date property for future reference.
Attachment #8518084 - Attachment is obsolete: true
Created attachment 8518772 [details] [diff] [review] add MsgDownload.jsm and use it
Attachment #8518109 - Attachment is obsolete: true
Created attachment 8687689 [details] [diff] [review] MsgDownload_jsm.patch Patch unbitrotted and const changed to var. Aceman, does this patch look complete as it is? I'm thinking of adding a reference to the message to jump to the message from download item and maybe also adding the subject. But this is something for a followup bug.
Comment on attachment 8687689 [details] [diff] [review] MsgDownload_jsm.patch I see now with this patch the saved attachments aren't shown in the list after a restart of TB (removing the patch and opening TB shows them again).
Does anybody know what this does? Is MsgDownloads.jsm some new code from m-c that migrates us more to the new downloads backend used in FF?
Not sure. A brief glance doesn't suggest it's a migrator - looks more like mail-specific utility functions that wrap Downloads.jsm. Hiro can probably tell us for sure.
Flags: needinfo?(mconley) → needinfo?(hiikezoe)
Yes, Downloads.jsm has a capability which stores additional information. Sender name is an example. Other examples: * sent date: it does not equal to saved date. * message subject: if the subject is shown in download list, user can easily find the original message which contained the attachment file, or find other attachment files in the same message.
You need to log in before you can comment on or make changes to this bug.