109134 - optimize mailHeaders.css to improve message display

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Message Display, defect)

Description

[Scott MacGregor]

I'm sure there are some style rules which can be written differently in
mailHeaders.css that will help reduce style resolution time which is a hot spot
during message display.

Comment 1

[Scott MacGregor]

accepting

Comment 2

[Scott MacGregor]

Attachment: not many style rules written incorrectly...but here's the optmization.

Comment 3

[Blake Ross]

Are you sure this is faster? Why?  Without the label identifier, the class can
apply to any element.

Comment 4

[Blake Ross]

Okay, Joe thinks it's faster, but please make sure things still look okay with
the message header in classic and modern. While doing the xul syntax changes, I
seem to recall |label| being needed for some odd reason, else things either
weren't bold or had the wrong margin.

Comment 5

[Scott MacGregor]

hyatt sayz...."just class names are faster because we look that up first". If
you specify a tag as well then after the class look up we have to check and make
sure the tag matches too.

Comment 6

[David :Bienvenu]

Comment on attachment 57099 [details] [diff] [review]
not many style rules written incorrectly...but here's the optmization.

r=bienvenu

Comment 7

[Scott MacGregor]

Comment on attachment 57099 [details] [diff] [review]
not many style rules written incorrectly...but here's the optmization.

i got a rs=sspitzer on this

Comment 8

[Scott MacGregor]

fix checked in.

Comment 9

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

I've verified that message display speed has improved (pretty nicely, I might 
add).  Too many good fixes landed in the same time frame for me to single out 
which improved it specifically.  See 
http://www.mozilla.org/mailnews/win_performance_results.html for more details.

Verified FIXED.