* **Electron Version** (output of `node_modules/.bin/electron --version`):
  * 4.0.4
* **Operating System** (Platform and Version):
  * macOS 10.14.3
* **Last known working Electron version** (if applicable):
  * unknown

### Expected Behavior
Setting a MenuItem to `visible: false` will hide it from the menu but **not** disable it.

See documentation https://electronjs.org/docs/api/menu-item

### Actual behavior
MenuItem is hidden AND disabled

### To Reproduce
https://github.com/aaronraimist/electron-quick-start/tree/visible-false

There should be one menu item visible (zoomin) and zoom out should also work but it doesn't.
(my issue is with zoomin but since that is kind of broken I did the demo with zoomout)

Menu code looks like
```
const template = [
  {
    label: 'View',
    submenu: [
	  { role: 'zoomin' },
      { role: 'zoomout', visible: false },
    ],
  }
]
```

```sh
# Clone this repository
git clone https://github.com/electron/electron-quick-start
# Go into the repository
cd electron-quick-start
# Checkout branch
git checkout visible-false
# Install dependencies
npm install
# Run the app
npm start
```

### Additional Information

I'm trying to work around https://github.com/electron/electron/issues/15496 by creating two menu items, one visible and one invisible but it doesn't work. See https://github.com/vector-im/riot-web/pull/8381
atom/browser/api/atom_api_menu.cc
v8::Local<v8::FunctionTemplate> prototype) {
  prototype->SetClassName(mate::StringToV8(isolate, "Menu"));
  mate::ObjectTemplateBuilder(isolate, prototype->PrototypeTemplate())
      .MakeDestroyable()
      .SetMethod("insertItem", &Menu::InsertItemAt)
      .SetMethod("insertCheckItem", &Menu::InsertCheckItemAt)
      .SetMethod("insertRadioItem", &Menu::InsertRadioItemAt)
      .SetMethod("insertSeparator", &Menu::InsertSeparatorAt)
      .SetMethod("insertSubMenu", &Menu::InsertSubMenuAt)
      .SetMethod("setIcon", &Menu::SetIcon)
      .SetMethod("setSublabel", &Menu::SetSublabel)
      .SetMethod("setRole", &Menu::SetRole)
      .SetMethod("clear", &Menu::Clear)
      .SetMethod("getIndexOfCommandId", &Menu::GetIndexOfCommandId)
      .SetMethod("getItemCount", &Menu::GetItemCount)
      .SetMethod("getCommandIdAt", &Menu::GetCommandIdAt)
      .SetMethod("getLabelAt", &Menu::GetLabelAt)
      .SetMethod("getSublabelAt", &Menu::GetSublabelAt)
      .SetMethod("getAcceleratorTextAt", &Menu::GetAcceleratorTextAt)
      .SetMethod("isItemCheckedAt", &Menu::IsItemCheckedAt)
      .SetMethod("isEnabledAt", &Menu::IsEnabledAt)
      .SetMethod("isVisibleAt", &Menu::IsVisibleAt)
      .SetMethod("popupAt", &Menu::PopupAt)
      .SetMethod("closePopupAt", &Menu::ClosePopupAt);
}
}  
}  
namespace {
using atom::api::Menu;
void Initialize(v8::Local<v8::Object> exports,