diff options
| author | evan@chromium.org <evan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-02-04 01:16:12 +0000 |
|---|---|---|
| committer | evan@chromium.org <evan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-02-04 01:16:12 +0000 |
| commit | e9ef1db24e1468bc378dffa650a0e56c794ee761 (patch) | |
| tree | 46f1e10eae2eaf5c4ba2bb895f37768db468b9e1 /chrome/browser/tab_contents/render_view_context_menu_gtk.cc | |
| parent | 46c74e569cb53ab4e283fa08d9938282a9a21a1e (diff) | |
| download | chromium_src-e9ef1db24e1468bc378dffa650a0e56c794ee761.zip chromium_src-e9ef1db24e1468bc378dffa650a0e56c794ee761.tar.gz chromium_src-e9ef1db24e1468bc378dffa650a0e56c794ee761.tar.bz2 | |
gtk: simplify menuing API
Bug 71821 looks like it was caused by using the wrong menu API, but
it wasn't obvious because there were so many entry points. This
change simplifies it to two main entry points -- one for context
menus (which belong at the mouse position) and one for button-initiated
menus (which belong alongside the button, like the wrench menu).
This also makes the right-click menu in the task manager come up on
mouse-down on GTK, as that is how right-click menus should work.
BUG=71821
TEST=bring up a notification, click the wrench, pick an item from
the submenu; the selection should take effect
Review URL: http://codereview.chromium.org/6250145
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@73726 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/tab_contents/render_view_context_menu_gtk.cc')
| -rw-r--r-- | chrome/browser/tab_contents/render_view_context_menu_gtk.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/browser/tab_contents/render_view_context_menu_gtk.cc b/chrome/browser/tab_contents/render_view_context_menu_gtk.cc index dc7b783..5478d6f 100644 --- a/chrome/browser/tab_contents/render_view_context_menu_gtk.cc +++ b/chrome/browser/tab_contents/render_view_context_menu_gtk.cc @@ -44,7 +44,7 @@ void RenderViewContextMenuGtk::Popup(const gfx::Point& point) { RenderWidgetHostView* rwhv = source_tab_contents_->GetRenderWidgetHostView(); if (rwhv) rwhv->ShowingContextMenu(true); - menu_gtk_->PopupAsContextAt(triggering_event_time_, point); + menu_gtk_->PopupAsContext(point, triggering_event_time_); } void RenderViewContextMenuGtk::StoppedShowing() { |