[webkit-reviews] review denied: [Bug 9984] ASSERTION FAILURE: _private->mouseDownEvent != nil (WebKit/WebView/WebHTMLView.m:4863 -[WebHTMLView(WebInternal) _delegateDragSourceActionMask]) : [Attachment 9570] Patch, including change log and a manual test

[bugzilla-request-daemon at opendarwin.org]

John Sullivan <sullivan at apple.com> has denied John Sullivan
<sullivan at apple.com>'s request for review:
Bug 9984: ASSERTION FAILURE: _private->mouseDownEvent != nil
(WebKit/WebView/WebHTMLView.m:4863 -[WebHTMLView(WebInternal)
_delegateDragSourceActionMask])
http://bugzilla.opendarwin.org/show_bug.cgi?id=9984

Attachment 9570: Patch, including change log and a manual test
http://bugzilla.opendarwin.org/attachment.cgi?id=9570&action=edit

------- Additional Comments from John Sullivan <sullivan at apple.com>
I don't understand how this patch would cause the assert not to fire. Formerly,
the assert would only be checked when self was not topHTMLView. But the patch
only affects the case when self is topHTMLView. So it seems that for any case
in which the assertion was formerly firing, the assertion will still fire. Am I
missing something?