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

Unified Diff: chrome/browser/gtk/menu_gtk.h

Issue 28105: Implement accelerators in the gtk port. (Closed)
Patch Set: Fixes for evanm Created 11 years, 10 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/gtk/menu_gtk.h
diff --git a/chrome/browser/gtk/menu_gtk.h b/chrome/browser/gtk/menu_gtk.h
index bef318ff11de4deaebef7acf1579caaba9716956..6dd1d05ae8a3a6916c9718d81680865eea92f087 100644
--- a/chrome/browser/gtk/menu_gtk.h
+++ b/chrome/browser/gtk/menu_gtk.h
@@ -38,7 +38,8 @@ class MenuGtk {
// Builds a MenuGtk that uses |delegate| to perform actions and |menu_data|
// to create the menu.
- MenuGtk(MenuGtk::Delegate* delegate, const MenuCreateMaterial* menu_data);
+ MenuGtk(MenuGtk::Delegate* delegate, const MenuCreateMaterial* menu_data,
+ GtkAccelGroup* accel_group);
// Builds a MenuGtk that uses |delegate| to create the menu as well as perform
// actions.
explicit MenuGtk(MenuGtk::Delegate* delegate);
@@ -84,6 +85,12 @@ class MenuGtk {
// Queries this object about the menu state.
MenuGtk::Delegate* delegate_;
+ // Accelerator group to add keyboard accelerators to.
+ GtkAccelGroup* accel_group_;
+
+ // The window this menu is attached to.
+ GtkWindow* window_;
+
// gtk_menu_popup() does not appear to take ownership of popup menus, so
// MenuGtk explicitly manages the lifetime of the menu.
GtkWidget* menu_;

Powered by Google App Engine
This is Rietveld 408576698