[webkit-reviews] review requested: [Bug 13947] Finish autogenerating the JS DOM traversal code : [Attachment 14798] patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed May 30 23:34:07 PDT 2007


Sam Weinig <sam at webkit.org> has asked  for review:
Bug 13947: Finish autogenerating the JS DOM traversal code
http://bugs.webkit.org/show_bug.cgi?id=13947

Attachment 14798: patch
http://bugs.webkit.org/attachment.cgi?id=14798&action=edit

------- Additional Comments from Sam Weinig <sam at webkit.org>
I am a little uneasy about this patch and wouldn't mind some harsh reviewing of
it.  The part I am uneasy about is the changing of the NodeFilter parameter of
Document::createNodeIterator and Document::createTreeWalker to be regular
pointers instead of PassRefPtrs.  I did this becuase it makes generating a lot
easier and that is how parameters are generally passed for the bindings.  If
there is any slowdown or other issues that this might cause I am sure I can
come up with a work around in the bindings scripts, but I thought I would check
if this way was OK first.



More information about the webkit-reviews mailing list