* Electron version: 1.6.3
* Operating system: Linux

### Expected behavior

When registering a `beforeunload` handler which cancels unloading, I would expect the handler to be called for every `BrowserWindow.reload()`.

### Actual behavior

On 1.6.3, the `beforeunload` handler is called for the first `reload` but not on subsequent calls. Even after removing the event handler, `reload` will not reload the window anymore.

### How to reproduce

```javascript
const cancelUnload = e => {
   console.log('cancelUnload called')
   e.returnValue = false
}

window.addEventListener('beforeunload', cancelUnload)

remote.getCurrentWindow().reload()  // [1] -> cancelUnload called
remote.getCurrentWindow().reload()  // [2] -> cancelUnload called

remote.getCurrentWindow().close()   // [3] -> cancelUnload called
remote.getCurrentWindow().close()   // [4] -> cancelUnload called

window.removeEventListener('beforeunload', cancelUnload)

remote.getCurrentWindow().reload()  // [5] -> reloads window

remote.getCurrentWindow().close()   // [6] -> closes window
```
I would expect `cancelUnload` to be printed to the console four times and the step at `[5]` to actually reload the window. This is works up to Electron 1.6.2.

On Electron 1.6.3 `cancelUnload` is not printed at step `[2]` (nor on any subsequent calls to reload). What's more, event after removing the `beforeunload` handler, `reload` does not work anymore: at step `[5]` the window does not reload (but closing the window at `[6]` works).
atom/browser/api/atom_api_web_contents.cc
.SetProperty("session", &WebContents::Session)
      .SetProperty("hostWebContents", &WebContents::HostWebContents)
      .SetProperty("devToolsWebContents", &WebContents::DevToolsWebContents)
      .SetProperty("debugger", &WebContents::Debugger);
}
AtomBrowserContext* WebContents::GetBrowserContext() const {
  return static_cast<AtomBrowserContext*>(web_contents()->GetBrowserContext());
}
void WebContents::OnRendererMessage(const base::string16& channel,
                                    const base::ListValue& args) {
  
  Emit(base::UTF16ToUTF8(channel), args);
}
void WebContents::OnRendererMessageSync(const base::string16& channel,
                                        const base::ListValue& args,
                                        IPC::Message* message) {
  
  EmitWithSender(base::UTF16ToUTF8(channel), web_contents(), message, args);
}
mate::Handle<WebContents> WebContents::CreateFrom(
    v8::Isolate* isolate, content::WebContents* web_contents) {
  
  auto existing = TrackableObject::FromWrappedClass(isolate, web_contents);
  if (existing)
    return mate::CreateHandle(isolate, static_cast<WebContents*>(existing));
  
  return mate::CreateHandle(isolate, new WebContents(isolate, web_contents,
        REMOTE));
}
mate::Handle<WebContents> WebContents::CreateFrom(