* [x] I have read the [Contributing Guidelines](https://github.com/electron/electron/blob/master/CONTRIBUTING.md) for this project.
* [x] I agree to follow the [Code of Conduct](https://github.com/electron/electron/blob/master/CODE_OF_CONDUCT.md) that this project adheres to.
* [x] I have searched the issue tracker for an issue that matches the one I want to file, without success.

### Issue Details

* **Electron Version:** 6.0.0-beta.13
* **Operating System:** Windows 10

### Expected Behavior

When calling `browserWindow.setAlwaysOnTop(true)`, I expect the `always-on-top-changed` event to be called with the boolean `isAlwaysOnTop` parameter set to `true`, and vice versa.

### Actual Behavior

I experience the opposite of above: When calling `browserWindow.setAlwaysOnTop(true)`, my `always-on-top-changed` handler is called with `false`, and vice versa.

### To Reproduce

I don't have any JS code to share and I'm not a JS dev (I use F# and compile to JS with Fable). I think the issue should be easy to reproduce though.
shell/browser/native_window_views.cc
#if defined(OS_WIN)
  DWORD affinity = enable ? WDA_MONITOR : WDA_NONE;
  ::SetWindowDisplayAffinity(GetAcceleratedWidget(), affinity);
#endif
}
void NativeWindowViews::SetFocusable(bool focusable) {
#if defined(OS_WIN)
  LONG ex_style = ::GetWindowLong(GetAcceleratedWidget(), GWL_EXSTYLE);
  if (focusable)
    ex_style &= ~WS_EX_NOACTIVATE;
  else
    ex_style |= WS_EX_NOACTIVATE;
  ::SetWindowLong(GetAcceleratedWidget(), GWL_EXSTYLE, ex_style);
  SetSkipTaskbar(!focusable);
  Focus(false);
#endif
}
void NativeWindowViews::SetMenu(AtomMenuModel* menu_model) {
#if defined(USE_X11)
  if (menu_model == nullptr) {
    global_menu_bar_.reset();
    root_view_->UnregisterAcceleratorsWithFocusManager();
    return;
  }
  if (!global_menu_bar_ && ShouldUseGlobalMenuBar())
    global_menu_bar_.reset(new GlobalMenuBarX11(this));
  
  if (global_menu_bar_ && global_menu_bar_->IsServerStarted()) {
    root_view_->RegisterAcceleratorsWithFocusManager(menu_model);