Summary: | [GTK] Context menu is hidden right after showing it when first menu item is not disabled | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Carlos Garcia Campos <cgarcia> | ||||
Component: | WebKit2 | Assignee: | Nobody <webkit-unassigned> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | gustavo, pnormand | ||||
Priority: | P2 | Keywords: | Gtk | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | PC | ||||||
OS: | Linux | ||||||
Attachments: |
|
Description
Carlos Garcia Campos
2011-12-29 08:18:52 PST
Created attachment 120751 [details]
Patch
Ah!, and the reason why it works when the first item is not disabled is because when the timeout is passed, the button release event is also ignored if the first menu item (that receives the release event) is not sentitive. Comment on attachment 120751 [details] Patch 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. (In reply to comment #3) > (From update of attachment 120751 [details]) > 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. It's an existing comment, just copied, see: http://trac.webkit.org/browser/trunk/Source/WebKit2/UIProcess/WebPageProxy.h#L600 And it means that those methods are the implementation of the WebPopupMenuProxy::Client virtual methods. Committed r103802: <http://trac.webkit.org/changeset/103802> |