500669 - indexing status & activity manager: special case "indexing one message"

Status: RESOLVED FIXED

(MailNews Core :: Database, defect)

Description

[David Ascher (:davida)]

gloda often ends up displaying messages in the status bar like "indexing 0 of 1 message".  Also, the activity manager shows very brief progress bars for these cases.

I propose:

1) the "indexing N of M" message starts with N=1
2) if M is 1, then we say something like "indexing message from <author>"
3) we don't create activity manager progress items for batches of 1.

Comment 1

[David Ascher (:davida)]

1) and 3).

Comment 2

[Blake Winton (:bwinton) (:☕️)]

Attachment: A patch for #1 and #3.

Comment 3

[Andrew Sutherland [:asuth] (he/him)]

Comment on attachment 394980 [details] [diff] [review]
A patch for #1 and #3.

suggest losing the bug number reference in the comment since there is little additional information to be found in this bug.

Comment 4

[Blake Winton (:bwinton) (:☕️)]

Attachment: The previous patch, with the bug number removed from the comment.

Comment 5

[Blake Winton (:bwinton) (:☕️)]

I believe this is a small enough change to not require ui-r or sr, so I'm marking it checkin-needed without those.  Please let me know if that's not the case, or if there's some other reason it shouldn't be checked in yet.

Thanks,
Blake.

Comment 6

[David :Bienvenu]

patches to TB code don't require sr...

Comment 7

[Mark Banner (:standard8)]

Checked in: http://hg.mozilla.org/comm-central/rev/7369d04a6047