[webkit-reviews] review denied: [Bug 69859] [Qt][WK2] ASSERT when displaying context menu in input field : [Attachment 110554] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Oct 14 01:48:32 PDT 2011
Simon Hausmann <hausmann at webkit.org> has denied Dinu Jacob
<dinu.jacob at Nokia.com>'s request for review:
Bug 69859: [Qt][WK2] ASSERT when displaying context menu in input field
https://bugs.webkit.org/show_bug.cgi?id=69859
Attachment 110554: Patch
https://bugs.webkit.org/attachment.cgi?id=110554&action=review
------- Additional Comments from Simon Hausmann <hausmann at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=110554&action=review
> Source/WebKit2/UIProcess/qt/WebContextMenuProxyQt.cpp:115
> - static_cast<QObject*>(subMenu.get())->setParent(menu.get());
> + subMenu->setParent(menu.get());
Could you explain (for example in the changelog or whereever appropriate) why
your patch fixes the assert?
Please also mention what assert exactly :)
More information about the webkit-reviews
mailing list