[Webkit-unassigned] [Bug 33696] let's cache nodelists instead of dynamicnodelist::cache

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=33696





--- Comment #73 from anton muhin <antonm at chromium.org>  2010-03-29 09:32:52 PST ---
(In reply to comment #68)
> (In reply to comment #64)
> > (In reply to comment #50)
> > > I had one other idea. It might be better for performance if the rare data bit
> > > check was inlined. We should do that in a follow-up patch.
> > 
> > Is not it inlined already (if you mean hasRareData())?
> 
> In JSNode::markChildren, we make a function call to markCachedNodeLists. I was
> looking for a code change to make sure it markChildren makes no function call
> in the common case where hasRareData() is false.

I see, thanks.  Let me give it a try.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.