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

[bugzilla-daemon at webkit.org]

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





--- Comment #5 from Darin Adler <darin at apple.com>  2010-01-15 12:08:13 PST ---
(From update of attachment 46659)
This isn’t just a performance optimization. It’s a behavior change. Caching the
node list object itself rather than just caching the underlying data means that
custom JavaScript properties attached by one client will be seen by the other
client. The client can tell when the cache is flushed by when the property does
or does not go away. I don’t know if this is desirable, but I do know that we
need test cases to cover it.

Please do not land this patch until we explore this issue.

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