### 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

It would appear that `app.focus()` stopped working in Electron 6, and remains broken in Electron 8.

* **Electron Version:**
  * 8.0.3
* **Operating System:**
  * macOS
* **Last Known Working Electron version:**
  * 5.0.13

### Expected Behavior
`app.focus()` should bring the application to the foreground.

### Actual Behavior
`app.focus()` does nothing.

### To Reproduce

Fiddle: https://gist.github.com/a772265b312329e3f3bc1feb7b6aff7d

1) Start this fiddle.
2) As soon as the window pops up, select another application so that Electron is in the background.
3) Within 5 seconds, the app will attempt to focus itself to no avail.

shell/browser/browser_win.cc
SHAddToRecentDocs(SHARD_APPIDINFO, nullptr);
}
void Browser::SetAppUserModelID(const base::string16& name) {
  electron::SetAppUserModelID(name);
}
bool Browser::SetUserTasks(const std::vector<UserTask>& tasks) {
  JumpList jump_list(GetAppUserModelID());
  if (!jump_list.Begin())
    return false;
  JumpListCategory category;
  category.type = JumpListCategory::Type::TASKS;
  category.items.reserve(tasks.size());
  JumpListItem item;
  item.type = JumpListItem::Type::TASK;
  for (const auto& task : tasks) {
    item.title = task.title;
    item.path = task.program;
    item.arguments = task.arguments;
    item.icon_path = task.icon_path;
    item.icon_index = task.icon_index;
    item.description = task.description;
    item.working_dir = task.working_dir;
    category.items.push_back(item);
  }
  jump_list.AppendCategory(category);
  return jump_list.Commit();
}
bool Browser::RemoveAsDefaultProtocolClient(const std::string& protocol,