[webkit-reviews] review requested: [Bug 5264]
document.createElementNS() should not allow to insert a
second <html> element : [Attachment 8328] Different approach
bugzilla-request-daemon at opendarwin.org
bugzilla-request-daemon at opendarwin.org
Mon May 15 13:37:07 PDT 2006
Rob Buis <rwlbuis at xs4all.nl> has asked for review:
Bug 5264: document.createElementNS() should not allow to insert a second <html>
element
http://bugzilla.opendarwin.org/show_bug.cgi?id=5264
Attachment 8328: Different approach
http://bugzilla.opendarwin.org/attachment.cgi?id=8328&action=edit
------- Additional Comments from Rob Buis <rwlbuis at xs4all.nl>
Hi Darin,
I think it will be hard to merge some of the functions. For instance
HTMLElement::childAllowed
really seems to be useful only for parsing, not for dom usage. In the new patch
I move the two checks
from Document::childallowed to Document::childTypeAllowed. That means for
calling childAllowed
on Document it behaves as before, for Document::childTypeAllowed it adds the
two checks.
The changed test results are official dom cases that now pass, except
documentnormalizedocument10.xhtml, which also "fails" in firefox. I think that
test is broken though,
the others probably need updated test results.
Firefox behaves correctly, it throws the exception.
Win IE does not run the testcase, it says:
Line 3
Char 3
Error: Object doesn't support this property or method
Cheers,
Rob.
More information about the webkit-reviews
mailing list