[webkit-reviews] review granted: [Bug 75357] [GTK] Context menu is hidden right after showing it when first menu item is not disabled : [Attachment 120751] Patch

[bugzilla-daemon at webkit.org]

Martin Robinson <mrobinson at webkit.org> has granted Carlos Garcia Campos
<cgarcia at igalia.com>'s request for review:
Bug 75357: [GTK] Context menu is hidden right after showing it when first menu
item is not disabled
https://bugs.webkit.org/show_bug.cgi?id=75357

Attachment 120751: Patch
https://bugs.webkit.org/attachment.cgi?id=120751&action=review

------- Additional Comments from Martin Robinson <mrobinson at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=120751&action=review


Nice!

> Source/WebKit2/UIProcess/WebPageProxy.h:595
> +    // WebPopupMenuProxy::Client

If you leave a comment here, I think it should be a full  sentence here.
"Called by WebPopupMenuProxy::Client." for instance.