Open Bug 1733504 Opened 4 years ago Updated 4 years ago

nsIMsgFolder.fetchMsgPreviewText should be async always

Categories

(MailNews Core :: General, task)

Tracking

(Not tracked)

People

(Reporter: benc, Unassigned)

Details

(Keywords: leave-open)

Attachments

(1 file)

nsIMsgFolder.fetchMsgPreviewText() takes a list of message keys, calculates a suitable string snippet to represent the message, and sets "preview" data on each messages nsIMsgDBHdr.
It's implemented by local and IMAP folders.
For locally-available messages (local folders and IMAP with offline copies of a message) this is done synchronously. But for online-only IMAP messages, it has to be async, to allow IMAP to go off and fetch enough of the messages to craft the preview.

It should be simplified by making it async always (even if the listener callback is invoked immediately).

Keywords: leave-open
Pushed by mkmelin@iki.fi: https://hg.mozilla.org/comm-central/rev/50c31d90adf5 Remove unused aLocalOnly param from nsIMsgFolder.fetchMsgPreviewText(). r=mkmelin
Assignee: benc → nobody
Status: ASSIGNED → NEW
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: