[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 #41 from anton muhin <antonm at chromium.org>  2010-03-26 10:28:31 PST ---
(In reply to comment #40)
> (In reply to comment #39)
> > Sure, I'd give it a try.  I am only concerned if we'd like to keep this
> > reference forever---it might be fine to GC it if it's not referenced anymore. 
> > But up to you.
> 
> The usual rule is that we don't want behavior to change based on when GC
> happens. So if you can add a custom property we want it to still be there later
> when you look for it.
> 
> When the node itself goes away, the node list will go away, so that's no real
> problem.
> 
> Once we start caching the node lists I think we need to go all the way and do
> the right thing with custom properties too.

Got it, thanks a lot.  Sending a patch (hopefully) soon.

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