Bug 54996 - [Qt] [WK2] When the context menu is empty, we should still send the signal QWKPage::showContextMenu()
Summary: [Qt] [WK2] When the context menu is empty, we should still send the signal QW...
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P4 Enhancement
Assignee: Benjamin Poulain
Keywords: Qt, QtTriaged
Depends on:
Reported: 2011-02-22 13:54 PST by Benjamin Poulain
Modified: 2011-02-23 01:11 PST (History)
1 user (show)

See Also:

Patch (2.64 KB, patch)
2011-02-22 14:19 PST, Benjamin Poulain
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Benjamin Poulain 2011-02-22 13:54:44 PST
Currently, the signal QWKPage::showContextMenu() is sent only if the context menu is not empty. This can be a problem for users who want to extend the context menu because they get no notification in that case.

The signal should be sent in any case, with an empty menu if necessary.
Comment 1 Benjamin Poulain 2011-02-22 14:19:17 PST
Created attachment 83387 [details]
Comment 2 Andreas Kling 2011-02-22 14:22:55 PST
Comment on attachment 83387 [details]

Comment 3 WebKit Commit Bot 2011-02-23 01:11:35 PST
Comment on attachment 83387 [details]

Clearing flags on attachment: 83387

Committed r79421: <http://trac.webkit.org/changeset/79421>
Comment 4 WebKit Commit Bot 2011-02-23 01:11:40 PST
All reviewed patches have been landed.  Closing bug.