[Webkit-unassigned] [Bug 16489] WebKit does not support ElementTraversal specification

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue May 20 14:34:39 PDT 2008


http://bugs.webkit.org/show_bug.cgi?id=16489


eric at webkit.org changed:

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




------- Comment #2 from eric at webkit.org  2008-05-20 14:34 PDT -------
(From update of attachment 21260)
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!


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



More information about the webkit-unassigned mailing list