[webkit-reviews] review denied: [Bug 6427] <tref> element not implemented : [Attachment 7773] Better patch

[bugzilla-request-daemon at opendarwin.org]

Eric Seidel <macdome at opendarwin.org> has denied Eric Seidel
<macdome at opendarwin.org>'s request for review:
Bug 6427: <tref> element not implemented
http://bugzilla.opendarwin.org/show_bug.cgi?id=6427

Attachment 7773: Better patch
http://bugzilla.opendarwin.org/attachment.cgi?id=7773&action=edit

------- Additional Comments from Eric Seidel <macdome at opendarwin.org>
attach() is still the wrong way to do this.  As this won't handle dynamic
updates.  I think this would be "ok" to land as is, but it's better to just fix
it right the first time.

Also, ideally your patch would contain a diff of the test results you've
changed, including possibly a dynamicly updating test to show that you got href
changes working correctly (which is what my previous attach() complaint is
about).

Other than that, this looks great.  Wonderful work as always Rob.