104519 - improve message display performance by selectively notifying the header sink

Status: VERIFIED DUPLICATE of bug 22960

(SeaMonkey :: MailNews: Message Display, defect, P1)

Description

[(not reading, please use [e-mail] instead)]

improve message display performance by selectively notifying the header sink

depending on our view mode ("mail.show_headers"), we don't have to tell he 
header sink about all the headers.

except in view all mode, we only need to tell the sink about subject, cc, to, 
from, date, and, newsgroups.

there are many headers we don't need to tell the sink about.

here comes a patch.

Comment 1

[Scott MacGregor]

i already have a mailnews perf bug where i talk about doing this. This should be
duped against my bug thanks.

Comment 2

[(not reading, please use [e-mail] instead)]

Attachment: patch

Comment 3

[(not reading, please use [e-mail] instead)]

I also #if 0 out some PRTime code that wasn't being used.

Comment 4

[(not reading, please use [e-mail] instead)]

over to mscott.

mscott points out there are a few more headers we need to allow through, see 
gExpandedHeaderList in msgHdrViewOverlay.js

as far as the PRTime code goes, double check.  We already call HandleHeader() 
on the sink with the string we get back from GenerateDateString().

Comment 5

[(not reading, please use [e-mail] instead)]

Attachment: patch, v2.

Comment 6

[scottputterman]

Marking P1 because it's message display performance. If you don't think this fix
warrants this priority, then please change it.

Comment 7

[[e-mail] (gone - use [e-mail] instead)]

This bug has lived as bug 22960 for quite some time.  Any reason to keep them 
seperate?

Comment 8

[(not reading, please use [e-mail] instead)]

patch is now part of patch for #22960

*** This bug has been marked as a duplicate of 22960 ***

Comment 9

[[e-mail] (gone - use [e-mail] instead)]

verified dup