Chromium Code Reviews| Index: chrome/browser/ui/views/extensions/browser_action_overflow_menu_controller.cc |
| diff --git a/chrome/browser/ui/views/extensions/browser_action_overflow_menu_controller.cc b/chrome/browser/ui/views/extensions/browser_action_overflow_menu_controller.cc |
| index 826c117e7013d5a6ee7cff693c9fc4f6792fda49..e26037be9efe992e56c133b6dfd9e6155411b75d 100644 |
| --- a/chrome/browser/ui/views/extensions/browser_action_overflow_menu_controller.cc |
| +++ b/chrome/browser/ui/views/extensions/browser_action_overflow_menu_controller.cc |
| @@ -87,6 +87,11 @@ void BrowserActionOverflowMenuController::CancelMenu() { |
| menu_->Cancel(); |
| } |
| +bool BrowserActionOverflowMenuController::IsCommandEnabled(int id) const { |
| + BrowserActionView* view = (*views_)[start_index_ + id - 1]; |
| + return view->button()->IsEnabled(owner_->GetCurrentTabId()); |
|
not at google - send to devlin
2012/07/12 01:07:29
Same comment here and in the GTK code:
ExtensionT
Yoyo Zhou
2012/07/12 01:33:27
I seem to recall this changes the appearance of me
|
| +} |
| + |
| void BrowserActionOverflowMenuController::ExecuteCommand(int id) { |
| BrowserActionView* view = (*views_)[start_index_ + id - 1]; |
| owner_->OnBrowserActionExecuted(view->button()); |