[webkit-reviews] review denied: [Bug 16489] WebKit does not support ElementTraversal specification : [Attachment 21260] Patch + testcases

[bugzilla-daemon at webkit.org]

Eric Seidel <eric at webkit.org> has denied Vincent Ricard
<magic at magicninja.org>'s request for review:
Bug 16489: WebKit does not support ElementTraversal specification
http://bugs.webkit.org/show_bug.cgi?id=16489

Attachment 21260: Patch + testcases
http://bugs.webkit.org/attachment.cgi?id=21260&action=edit

------- Additional Comments from Eric Seidel <eric at webkit.org>
The implementation looks fine.

The test cases should be written as js test cases, following the form used by
so many in fast/js (and other places).	you can use make-js-test-wrappers to
generate the final .html files once you add your .js files.  (js test cases use
our shouldBe functions, etc.)

Also, these new functions on Element should be const.

Otherwise it looks fine to me.

I would r+ this but, AFAIK you don't have commit bit (so any edits you make
need to go through a posted patch).  So I'd like to see a patch with cleaner
test cases before r+ing.

Thanks!