diff options
| author | estade@chromium.org <estade@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-06 21:04:55 +0000 |
|---|---|---|
| committer | estade@chromium.org <estade@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-06 21:04:55 +0000 |
| commit | a8e91124532061d21f0114605530c605582144e8 (patch) | |
| tree | 4ee30fb79cdb064517c5ad0a1c11a75328a7db04 /chrome/browser/gtk/browser_actions_toolbar_gtk.cc | |
| parent | 4e59e814cc1264300b597c3d62a8ccb4da264889 (diff) | |
| download | chromium_src-a8e91124532061d21f0114605530c605582144e8.zip chromium_src-a8e91124532061d21f0114605530c605582144e8.tar.gz chromium_src-a8e91124532061d21f0114605530c605582144e8.tar.bz2 | |
GTK: Change menu delegate callback from CommandWasExecuted to CommandWillBeExecuted.
Both the menu itself and the delegate were being destroyed in ExecuteCommand, double whammy!
BUG=40477
TEST=disable browser action extension via context menu
Review URL: http://codereview.chromium.org/1593012
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@43757 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/gtk/browser_actions_toolbar_gtk.cc')
| -rw-r--r-- | chrome/browser/gtk/browser_actions_toolbar_gtk.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/browser/gtk/browser_actions_toolbar_gtk.cc b/chrome/browser/gtk/browser_actions_toolbar_gtk.cc index 1d25c2a..9087793 100644 --- a/chrome/browser/gtk/browser_actions_toolbar_gtk.cc +++ b/chrome/browser/gtk/browser_actions_toolbar_gtk.cc @@ -200,7 +200,7 @@ class BrowserActionButton : public NotificationObserver, gtk_util::GrabAllInput(toolbar_->overflow_menu_->widget()); } - virtual void CommandWasExecuted() { + virtual void CommandWillBeExecuted() { // If the context menu was showing for the overflow menu, and a command // is executed, then stop showing the overflow menu. if (toolbar_->overflow_menu_.get()) |