[webkit-reviews] review denied: [Bug 3439] On slow machines javascript doesn't behave as it should : [Attachment 7051] naive fix

[bugzilla-request-daemon at opendarwin.org]

Geoffrey Garen <ggaren at apple.com> has denied Geoffrey Garen
<ggaren at apple.com>'s request for review:
Bug 3439: On slow machines javascript doesn't behave as it should
http://bugzilla.opendarwin.org/show_bug.cgi?id=3439

Attachment 7051: naive fix
http://bugzilla.opendarwin.org/attachment.cgi?id=7051&action=edit

------- Additional Comments from Geoffrey Garen <ggaren at apple.com>
Yeah, you do need a test. And a ChangeLog. DumpRenderTree can simulate mouse
events, so a test is possible.

You can test oldUnder itself for truth instead of oldUnder.get().

Doesn't this patch prevent mouseout from firing when you move the mouse between
frames? In general, what's the purpose of the test

    oldUnder->getDocument() != frame()->document()

? To ensure that the node is still in the document? inDocument() is probably
better for that. (Make sure your test covers that case.)