Closed Bug 104519 Opened 23 years ago Closed 23 years ago

improve message display performance by selectively notifying the header sink

Categories

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

x86
Windows 2000
defect

Tracking

(Not tracked)

VERIFIED DUPLICATE of bug 22960
mozilla0.9.7

People

(Reporter: sspitzer, Assigned: mscott)

Details

(Keywords: perf)

Attachments

(1 file, 1 obsolete file)

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.
i already have a mailnews perf bug where i talk about doing this. This should be
duped against my bug thanks.
Attached patch patch (obsolete) — Splinter Review
I also #if 0 out some PRTime code that wasn't being used.
Status: NEW → ASSIGNED
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().

Assignee: sspitzer → mscott
Status: ASSIGNED → NEW
Attachment #53383 - Attachment is obsolete: true
Marking P1 because it's message display performance. If you don't think this fix
warrants this priority, then please change it.
Keywords: nsbeta1+
Priority: -- → P1
Target Milestone: --- → mozilla0.9.7
This bug has lived as bug 22960 for quite some time.  Any reason to keep them 
seperate?
Keywords: perf
QA Contact: esther → stephend
patch is now part of patch for #22960

*** This bug has been marked as a duplicate of 22960 ***
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → DUPLICATE
verified dup
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: