* Electron Version: 3.0.0-beta.1 ~ 3.0.0-beta.6
* Operating System (Platform and Version): Ubuntu 16.04.2 LTS
* Last known working Electron version: versions before 3.0.0-beta.1

**Expected Behavior**
Console prints out `time to print stuff` when `Ctrl+P` is pressed.
```
const {Menu, MenuItem} = require('electron')
const menu = new Menu()
  
app.on('ready', () => {
  createWindow()
  menu.append(new MenuItem({
    label: 'Print',
    accelerator: 'CmdOrCtrl+P',
    click: () => { console.log('time to print stuff') }
  }))
  Menu.setApplicationMenu(menu)
})
```
https://github.com/hackjutsu/electron-quick-start/commit/6ceb5d7dcd2d946c071b43a3fdd1b0434ddebff5
**Actual behavior**
No output. The accelerator event is not triggered.

**To Reproduce**
```sh
git clone https://github.com/hackjutsu/electron-quick-start
cd electron-quick-start
npm install
npm run start
```
Try pressing `Ctrl+P`, and observe the output. (nothing happen...)
Change the electron version back to 2.0.0 in `pakcage.json`
```sh
npm install
npm run start
```
Try pressing `Ctrl+P`, and observe the output. (the log `time to print stuff` will show up in console)

atom/browser/ui/views/root_view.cc
: window_(window),
      last_focused_view_tracker_(std::make_unique<views::ViewTracker>()) {
  set_owned_by_client();
}
RootView::~RootView() {}
void RootView::SetMenu(AtomMenuModel* menu_model) {
  if (menu_model == nullptr) {
    
    accelerator_table_.clear();
    GetFocusManager()->UnregisterAccelerators(this);
    
    SetMenuBarVisibility(false);
    menu_bar_.reset();
    return;
  }
  RegisterAccelerators(menu_model);
  
  if (!window_->has_frame())
    return;
  if (!menu_bar_) {
    menu_bar_.reset(new MenuBar(this));
    menu_bar_->set_owned_by_client();
    if (!menu_bar_autohide_)
      SetMenuBarVisibility(true);
  }
  menu_bar_->SetMenu(menu_model);
  Layout();
}
bool RootView::HasMenu() const {