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

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

Issue 159822: linux: fix regression where opening the page menu would cause a reload (Closed)
Patch Set: Created 11 years, 5 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 | chrome/browser/gtk/menu_gtk.cc » ('j') | chrome/browser/gtk/menu_gtk.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/gtk/menu_gtk.h
diff --git a/chrome/browser/gtk/menu_gtk.h b/chrome/browser/gtk/menu_gtk.h
index 7746af32cf1c93cf70ffc4123a66e232e3bd6015..b06c77cac86c186234dc20465d0143c51e8d690f 100644
--- a/chrome/browser/gtk/menu_gtk.h
+++ b/chrome/browser/gtk/menu_gtk.h
@@ -137,6 +137,11 @@ class MenuGtk {
// gtk_menu_popup() does not appear to take ownership of popup menus, so
// MenuGtk explicitly manages the lifetime of the menu.
OwnedWidgetGtk menu_;
+
+ // True when we should ignore "activate" signals. Used to prevent
+ // menu items from getting activated when we are setting up the
+ // menu.
+ static bool block_activation_;
};
#endif // CHROME_BROWSER_GTK_MENU_GTK_H_
« no previous file with comments | « no previous file | chrome/browser/gtk/menu_gtk.cc » ('j') | chrome/browser/gtk/menu_gtk.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698