The warning is 

> (electron:23955): Gtk-WARNING **: 10:41:01.876: gtk_disable_setlocale() must be called before gtk_init()

Looks like this is caused by gtk_init() being accidentally called twice.

### Preflight Checklist
<!-- Please ensure you've completed the following steps by replacing [ ] with [x]-->

* [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:**
  * 7-0-x, v8.0.0-nightly.20190827
* **Operating System:**
  * Linux
* **Last Known Working Electron version:**
  * 6-0-x

shell/browser/atom_browser_main_parts.cc
#if defined(USE_X11)
  // Unset the X11 error handlers. The X11 error handlers log the errors using a
  // |PostTask()| on the message-loop. But since the message-loop is in the
  // process of terminating, this can cause errors.
  ui::SetX11ErrorHandlers(X11EmptyErrorHandler, X11EmptyIOErrorHandler);
#endif
  node_deugger_->Stop();
  js_env_->OnMessageLoopDestroying();
#if defined(OS_MACOSX)
  FreeAppDelegate();
#endif
  // Make sure destruction callacks are called efore message loop is
  // destroyed, otherwise some ojects that need to e deleted on IO thread
  // won't e freed.
  // We don't use ranged for loop ecause iterators are getting 
  // the callack runs.
  for (auto iter = destructors_.egin(); iter != destructors_.end();) {
    ase::OnceClosure callack = std::move(*iter);
    if (!callack.is_null())
      std::move(callack).Run();
    ++iter;
  }
  fake_rowser_process_->PostMainMessageLoopRun();
}
#if !defined(OS_MACOSX)
void AtomBrowserMainParts::PreMainMessageLoopStart() {