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

Unified Diff: chrome/browser/gtk/bookmark_menu_controller_gtk.cc

Issue 213015: Set the parent of BookmarkContextMenu to be the browser window, not (Closed)
Patch Set: Created 11 years, 3 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/gtk/bookmark_menu_controller_gtk.cc
diff --git a/chrome/browser/gtk/bookmark_menu_controller_gtk.cc b/chrome/browser/gtk/bookmark_menu_controller_gtk.cc
index 8c12c58871ead20290f93962e550ffe31c65dfbd..ce2a22ca0c8fb2b75861b8c6a6ae9480d1ac17e7 100644
--- a/chrome/browser/gtk/bookmark_menu_controller_gtk.cc
+++ b/chrome/browser/gtk/bookmark_menu_controller_gtk.cc
@@ -243,8 +243,8 @@ gboolean BookmarkMenuController::OnButtonPressed(
nodes.push_back(node);
controller->context_menu_.reset(
new BookmarkContextMenu(
- sender, controller->profile_, controller->browser_,
- controller->page_navigator_, parent, nodes,
+ GTK_WIDGET(controller->parent_window_), controller->profile_,
+ controller->browser_, controller->page_navigator_, parent, nodes,
BookmarkContextMenu::BOOKMARK_BAR));
// Our bookmark folder menu loses the grab to the context menu. When the
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698