[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: Khtml too slow ?
From:       Dirk Mueller <mueller () kde ! org>
Date:       2000-09-26 15:31:06
[Download RAW message or body]

On Die, 26 Sep 2000, Lars Knoll wrote:

cc'ed to kfm-devel as this is the better place ;)

> Hmmm... You mean building up a seperate list for each element? 

Well, I'm not sure as I don't know all the details of CSS rules inheritance
yet. the performance hole is CSSStyleSelector::styleForElement which calls
CSSStyleSelector::collect which runs through a very long list of
priority-sorted style rules. 

I was thinking about changing it to a tree like structure so you don't have
to apply every existing rule to get a valid RenderStyle object but just copy
the parent one and apply the "new" rules..

i.e the list should be split in pieces and managed in the RenderStyle
"tree". But as I don't know the exact rules yet I cannot judge if this is a
good/working idea..

> That should 
> get it down to quite small numbers, and would probably accelerate parsing a 
> lot. Shouldn't be too hard to do.

sure, but I considered it more important to fix the parsing crashes we're
hitting lately to be more important, that's why I didn't post a mail about
improving the style selector yet. 


Dirk

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic