90% of time spent activating mail window is in ReResolveStyleContext

VERIFIED WONTFIX

Status

()

--
major
VERIFIED WONTFIX
18 years ago
16 years ago

People

(Reporter: mikepinkerton, Assigned: attinasi)

Tracking

({perf})

Trunk
PowerPC
Mac System 9.x
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

18 years ago
when profile the 'activate' phase of bringing a mail window to the fg, 90% of the 
time is spent in ReResolveStyleContext, probably because of use of :focus.

Note that this is easily twice the time spent than a similar profile of bringing 
a navigator window to the fg. It is very noticeable, even in optimized builds on 
my fast mac.
(Reporter)

Updated

18 years ago
Keywords: mozilla0.9, nsbeta1, nsmac1, perf
(Reporter)

Updated

18 years ago
Blocks: 48274

Comment 1

18 years ago
Probably a lot of attribute tweaking happening as a result of command updating 
here too.
Are you speaking about the mail 3-pane window?
(Reporter)

Comment 3

18 years ago
yes, the 3-pane mail window
(Assignee)

Comment 4

18 years ago
I can profile this, but if there is a lot of style resolution going on the most 
likely reason is that there are evil style rules (like *:active) or lots 
resolution getting triggered due to attribute or content changes. 

Please indicate what stylesheets apply here so I can look at the rules in 
addition to the profiling. Is this only on the Mac? If so, some profile data 
would be nice too since I cannot currently profile on the Mac.

Also, what is the 'bug' here? How much time is reasonable for resolving style? 
Thanks.
(Assignee)

Comment 5

18 years ago
Still looking for some information here... see my prior comment please.

Comment 6

18 years ago
Outliner will make this bug obsolete.  IMO this bug should just be closed as
WONTFIX, since the new tree widget solves this problem and will be landing
during the 0.8.1 freeze.
(Assignee)

Comment 7

18 years ago
I can buy that - Thanks David!

Marking WONTFIX since David's new outliner will make this obsolete. If the
reporter or anybody else disagrees with the resolution, please reopen and
provide mo' data.
Status: NEW → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → WONTFIX

Comment 8

18 years ago
The old tree widget has some skanky :focus rules that probably caused the
performance hit.
(Reporter)

Comment 9

18 years ago
you're _sure_ about that? totally _sure_?
v
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.