[Webkit-unassigned] [Bug 166059] [GTK] Crash on drag above web view of nautilus' tab

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=166059

--- Comment #2 from Milan Crha <mcrha at redhat.com> ---
Yes, I wouldn't suggest to check for `this != NULL`, illegal or not, it looks and feels awkward and like a very bad hack. What I've on my mind is to change the asserts into a real runtime check, instead of "check only in debug build, or whatever". That will make the code more robust. And if you feel that it'll hide some issues, then keep there both the assert and the runtime check, and eventually print a runtime warning if the runtime check fails, if you think that the DragAndDropHandler cannot be NULL in all cases.

In other words, the change should come, from my point of view, into webkitWebViewBaseDragDrop() and the related drag & drop functions, instead of into the DragAndDropHandler class.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20161221/0a2e2a17/attachment.html>