### Preflight Checklist
* [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:**
10.0.0-nightly.20200220
* **Operating System:**
macOS 10.15.1
* **Last Known Working Electron version:**
I don't know (judging by implementation it never worked)

### Expected Behavior:
Call to BrowserWindow.webContents.focus focuses proper window

### Actual Behavior:
Call  to BrowserWindow.webContents.focus does nothing

### To Reproduce:

Run following simple app:
```
const { app, BrowserWindow } = require('electron')

function createWindow () {
  let win1 = new BrowserWindow({ x:100, y: 300, width: 300, height: 200 })
  win1.on('closed', () => {
    win1 = null
  })
  win1.loadURL('https://github.com')

  let win2 = new BrowserWindow({ x:300, y: 300, width: 300, height: 200 })
  win2.on('closed', () => {
    win1 = null
  })
  win2.loadURL('http://www.example.com')

  setTimeout(() => {
    win1.webContents.focus();
  }, 3000);
}

app.on('ready', createWindow)
```

It works as expected on Windows: it creates two windows and after three seconds it focuses the first one. On mac it doesn't work and call to `webContents.focus` doesn't change the focus (it stays on the second window).
shell/browser/api/electron_api_web_contents.cc
if (options.Get("userAgent", &user_agent))
    ontents()->SetUserAgentOverride(user_agent, false);
  std::string extra_headers;
  if (options.Get("extraHeaders", &extra_headers))
    params.extra_headers = extra_headers;
  scoped_refptr<network::ResourceRequestBody> body;
  if (options.Get("postData", &body)) {
    params.post_data = body;
    params.load_type = content::NavigationController::LOAD_TYPE_HTTP_POST;
  }
  GURL base_url_for_data_url;
  if (options.Get("baseURLForDataURL", &base_url_for_data_url)) {
    params.base_url_for_data_url = base_url_for_data_url;
    params.load_type = content::NavigationController::LOAD_TYPE_DATA;
  }
  bool reload_ignoring_cache = false;
  if (options.Get("reloadIgnoringCache", &reload_ignoring_cache) &&
      reload_ignoring_cache) {
    params.reload_type = content::ReloadType::BYPASSING_CACHE;
  }
  // Calling LoadURLWithParams() can trigger JS which destroys |this|.
  auto weak_this = GetWeakPtr();
  params.transition_type = ui::PAGE_TRANSITION_TYPED;
  params.should_clear_history_list = true;
  params.override_user_agent = content::NavigationController::UA_OVERRIDE_TRUE;
  // Discord non-committed entries to ensure that we don't re-use a pending
  // entry
  ontents()->GetController().DiscardNonCommittedEntries();
  ontents()->GetController().LoadURLWithParams(params);
  // ⚠️WARNING!⚠️