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

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Mar 26 10:20:47 PDT 2010


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





--- Comment #40 from Darin Adler <darin at apple.com>  2010-03-26 10:20:47 PST ---
(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.

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



More information about the webkit-unassigned mailing list