| Index: views/controls/menu/menu_item_view.cc
|
| diff --git a/views/controls/menu/menu_item_view.cc b/views/controls/menu/menu_item_view.cc
|
| index 4b251dfab3f58ee8a5759e0230cdacce1bcb9485..8c0e07e4a88e9ca298d2ed140b4332a523493dc9 100644
|
| --- a/views/controls/menu/menu_item_view.cc
|
| +++ b/views/controls/menu/menu_item_view.cc
|
| @@ -36,7 +36,7 @@ namespace {
|
| class EmptyMenuMenuItem : public MenuItemView {
|
| public:
|
| explicit EmptyMenuMenuItem(MenuItemView* parent)
|
| - : MenuItemView(parent, 0, NORMAL) {
|
| + : MenuItemView(parent, 0, EMPTY) {
|
| // Set this so that we're not identified as a normal menu item.
|
| SetID(kEmptyMenuItemViewID);
|
| SetTitle(UTF16ToWide(
|
| @@ -136,6 +136,7 @@ void MenuItemView::GetAccessibleState(ui::AccessibleViewState* state) {
|
| break;
|
| case NORMAL:
|
| case SEPARATOR:
|
| + case EMPTY:
|
| // No additional accessibility states currently for these menu states.
|
| break;
|
| }
|
| @@ -260,6 +261,7 @@ MenuItemView* MenuItemView::AddMenuItemAt(int index,
|
| const std::wstring& label,
|
| const SkBitmap& icon,
|
| Type type) {
|
| + DCHECK_NE(type, EMPTY);
|
| DCHECK_LE(0, index);
|
| if (!submenu_)
|
| CreateSubmenu();
|
| @@ -563,9 +565,9 @@ void MenuItemView::Init(MenuItemView* parent,
|
| has_icons_ = false;
|
|
|
| // Don't request enabled status from the root menu item as it is just
|
| - // a container for real items.
|
| + // a container for real items. EMPTY items will be disabled.
|
| MenuDelegate* root_delegate = GetDelegate();
|
| - if (parent && root_delegate)
|
| + if (parent && type != EMPTY && root_delegate)
|
| SetEnabled(root_delegate->IsCommandEnabled(command));
|
| }
|
|
|
|
|