Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(715)

Unified Diff: chrome/browser/ui/gtk/tab_contents/chrome_web_contents_view_delegate_gtk.cc

Issue 130773004: Start moving context menu code to RenderFrame. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 6 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/gtk/tab_contents/chrome_web_contents_view_delegate_gtk.cc
===================================================================
--- chrome/browser/ui/gtk/tab_contents/chrome_web_contents_view_delegate_gtk.cc (revision 247964)
+++ chrome/browser/ui/gtk/tab_contents/chrome_web_contents_view_delegate_gtk.cc (working copy)
@@ -128,6 +128,7 @@
}
void ChromeWebContentsViewDelegateGtk::ShowContextMenu(
+ content::RenderFrameHost* render_frame_host,
const content::ContextMenuParams& params) {
// Find out the RenderWidgetHostView that corresponds to the render widget on
// which this context menu is showed, so that we can retrieve the last mouse
@@ -149,7 +150,7 @@
}
context_menu_.reset(
- new RenderViewContextMenuGtk(web_contents_, params, view));
+ new RenderViewContextMenuGtk(render_frame_host, params, view));
context_menu_->Init();
// Don't show empty menus.

Powered by Google App Engine
This is Rietveld 408576698