[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


Darin Adler <darin at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #51734|review?                     |review-
               Flag|                            |




--- Comment #36 from Darin Adler <darin at apple.com>  2010-03-26 09:26:54 PST ---
(From update of attachment 51734)
> -    [ "document.getElementsByTagName('body')" ], // NodeList
> +    [ "document.getElementsByTagName('body')", "allow custom skip" ], // NodeList

This says that before the patch we did not allow custom properties on node
lists. And after the patch we do allow custom properties on node lists. That
has nothing to do with caching. Why did this patch have any effect on this?
This makes no sense.

I understand that the caching mean this would change from "allow custom" to
"allow custom skip", but I don't see why this patch changes behavior of adding
a custom property to a node list.

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