[webkit-reviews] review requested: [Bug 12938] Google calendar settings page crashes : [Attachment 13783] Revised patch again

[bugzilla-daemon at webkit.org]

Marvin Decker <marv.decker at gmail.com> has asked  for review:
Bug 12938: Google calendar settings page crashes
http://bugs.webkit.org/show_bug.cgi?id=12938

Attachment 13783: Revised patch again
http://bugs.webkit.org/attachment.cgi?id=13783&action=edit

------- Additional Comments from Marvin Decker <marv.decker at gmail.com>
This is a revised patch that also fixes the potential crash on removing forms
in the document. This is indeed a problem: the pointer to the form is left in
the old document's hash table when it is moved.

As discussed above, I was having problems generating a test for the move
between documents bug. After spending several hours on this, I've never
actually seen this one cause a crash, although it's definitely possible (and
also a memory leak).

I can convert the testcase I attached at the top (first attachment) into a
regression test if you would like, but it would not be totally reliable since
it depends on the memory manager's feeling of where to place things. My
PowerBook seems to never trip on this page, although a colleague's Intel-based
iMac does after a bit. Requesting review to get official opinion on this.