Commit 22273a3d authored by Jeremy Apthorp's avatar Jeremy Apthorp Committed by Commit Bot

Fix submenu activation in AppIndicatorIconMenu

menu_model_ refers to the root model, so it was previously checking the
enabled state of the wrong menu model.

Change-Id: Ia1a3662ea68779cbb86663973ba4599d9fabb054
Reviewed-on: https://chromium-review.googlesource.com/c/1336443Reviewed-by: default avatarEvan Stade <estade@chromium.org>
Commit-Queue: Jeremy Apthorp <jeremya@chromium.org>
Cr-Commit-Position: refs/heads/master@{#608512}
parent ed63e54f
...@@ -116,7 +116,7 @@ void AppIndicatorIconMenu::OnMenuItemActivated(GtkWidget* menu_item) { ...@@ -116,7 +116,7 @@ void AppIndicatorIconMenu::OnMenuItemActivated(GtkWidget* menu_item) {
return; return;
// The menu item can still be activated by hotkeys even if it is disabled. // The menu item can still be activated by hotkeys even if it is disabled.
if (menu_model_->IsEnabledAt(id)) if (model->IsEnabledAt(id))
ExecuteCommand(model, id); ExecuteCommand(model, id);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment