[webkit-reviews] review granted: [Bug 8983] Autogenerate another 22
HTML classes : [Attachment 8398] Patch
bugzilla-request-daemon at opendarwin.org
bugzilla-request-daemon at opendarwin.org
Thu May 18 16:14:29 PDT 2006
Eric Seidel <macdome at opendarwin.org> has granted Eric Seidel
<macdome at opendarwin.org>'s request for review:
Bug 8983: Autogenerate another 22 HTML classes
http://bugzilla.opendarwin.org/show_bug.cgi?id=8983
Attachment 8398: Patch
http://bugzilla.opendarwin.org/attachment.cgi?id=8398&action=edit
------- Additional Comments from Eric Seidel <macdome at opendarwin.org>
In general this looks great.
We shoudl make sure to land a test for every place in which we differ from the
spec. In autogenerating other classes I found that the dom 2 html spec changed
the type of several attributes, which might be what you're noticing in these as
well.
Second, you also should fix the style Document *doc instead of Document* doc,
in HTMLBodyElement.cpp
With those additional test cases to cover the places where we differ from the
spec, you're good to land.
More information about the webkit-reviews
mailing list