* 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.h
content::NavigationHandle* navigation_handle) override;
  void DidFinishNavigation(
      content::NavigationHandle* navigation_handle) override;
  bool OnMessageReceived(const IPC::Message& message) override;
  void WebContentsDestroyed() override;
  void NavigationEntryCommitted(
      const content::LoadCommittedDetails& load_details) override;
  void TitleWasSet(content::NavigationEntry* entry, bool explicit_set) override;
  void DidUpdateFaviconURL(
      const std::vector<content::FaviconURL>& urls) override;
  void PluginCrashed(const base::FilePath& plugin_path,
                     base::ProcessId plugin_pid) override;
  void MediaStartedPlaying(const MediaPlayerInfo& video_type,
                           const MediaPlayerId& id) override;
  void MediaStoppedPlaying(const MediaPlayerInfo& video_type,
                           const MediaPlayerId& id) override;
  void DidChangeThemeColor(SkColor theme_color) override;
  
  void Observe(int type,
               const content::NotificationSource& source,
               const content::NotificationDetails& details) override;
  
  void DevToolsReloadPage() override;
  
  void DevToolsFocused() override;
  void DevToolsOpened() override;
  void DevToolsClosed() override;
 private:
  struct LoadURLParams {
    LoadURLParams() : params(GURL()), id(0) {}