[webkit-reviews] review cancelled: [Bug 3240] Support the frame and rules attributes on HTML tables : [Attachment 10585] Patch that implements support for the frame and rules attributes

[bugzilla-request-daemon at opendarwin.org]

Dave Hyatt <hyatt at apple.com> has cancelled Dave Hyatt <hyatt at apple.com>'s
request for review:
Bug 3240: Support the frame and rules attributes on HTML tables
http://bugzilla.opendarwin.org/show_bug.cgi?id=3240

Attachment 10585: Patch that implements support for the frame and rules
attributes
http://bugzilla.opendarwin.org/attachment.cgi?id=10585&action=edit

------- Additional Comments from Dave Hyatt <hyatt at apple.com>
Ok, I cleaned up the static casts and compacted that code.

I did not convert the member variables to RefPtrs.  I started to do so and
realized that the code surrounding the caching of these variables is (right
now) completely buggy (only the body member is actually being ref'ed right
now).  Because the problem is more complicated (and not relevant at all to this
patch),  I'm filing a followup bug about making the management of those
pointers better (both RefPtr conversion and then making them update properly as
well).