* **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.h
protected:
  Menu(v8::Isolate* isolate, v8::Local<v8::Object> wrapper);
  ~Menu() override;
  
  void AfterInit(v8::Isolate* isolate) override;
  
  bool IsCommandIdChecked(int command_id) const override;
  bool IsCommandIdEnabled(int command_id) const override;
  bool IsCommandIdVisible(int command_id) const override;
  bool GetAcceleratorForCommandIdWithParams(
      int command_id,
      bool use_default_accelerator,
      ui::Accelerator* accelerator) const override;
  bool ShouldRegisterAcceleratorForCommandId(int command_id) const override;
  void ExecuteCommand(int command_id, int event_flags) override;
  void OnMenuWillShow(ui::SimpleMenuModel* source) override;
  virtual void PopupAt(TopLevelWindow* window,
                       int x,
                       int y,
                       int positioning_item,
                       const base::Closure& callback) = 0;
  virtual void ClosePopupAt(int32_t window_id) = 0;
  std::unique_ptr<AtomMenuModel> model_;
  Menu* parent_ = nullptr;
  
  void OnMenuWillClose() override;
  void OnMenuWillShow() override;
 private:
  void InsertItemAt(int index, int command_id, const base::string16& label);