[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