[webkit-reviews] review denied: [Bug 3298] insertRow generates DOM
Exception if TABLE does not possess a TBODY :
[Attachment 2672] Update firstBody accordingly
bugzilla-request-daemon at opendarwin.org
bugzilla-request-daemon at opendarwin.org
Mon Jun 27 17:17:25 PDT 2005
Darin Adler <darin at apple.com> has denied Anders Carlsson <andersca at mac.com>'s
request for review:
Bug 3298: insertRow generates DOM Exception if TABLE does not possess a TBODY
http://bugzilla.opendarwin.org/show_bug.cgi?id=3298
Attachment 2672: Update firstBody accordingly
http://bugzilla.opendarwin.org/attachment.cgi?id=2672&action=edit
------- Additional Comments from Darin Adler <darin at apple.com>
I think hasChildElements belongs in either NodeImpl or ElementImpl rather than
here.
No need to check hasChildNodes() in hasChildElement(). firstChild() will be 0,
and the function will quickly return false without the extra check.
I think a better approach to tracking firstBody would be to hold a ref to
firstBody, then check if its parent is still "this" in childrenChanged, and
deref it if so. Also deref in the destructor of course. No need to override
removeChild.
More information about the webkit-reviews
mailing list