Use exec instead of execSync and don't exit

execSync on Windows causes a hang and app.exit() will have data loss because it will bypass normal shutdown

app/index.js

@@ -92,7 +92,6 @@ app.commandLine.appendSwitch('enable-features', 'BlockSmallPluginContent,PreferH
 // Fix https://github.com/brave/browser-laptop/issues/15337
 app.commandLine.appendSwitch('disable-databases')
 
-
 // Domains to accept bad certs for. TODO: Save the accepted cert fingerprints.
 let acceptCertDomains = {}
 let errorCerts = {}

js/stores/appStore.js

@@ -203,17 +203,16 @@ const launchBraveCore = (state) => {
   }
 
   const childProcess = require('child_process')
-  const execSync = childProcess.execSync
+  const exec = childProcess.exec
 
   try {
     if (platformUtil.isDarwin()) {
-      execSync(`open -a "${installedPath}"`)
+      exec(`open -a "${installedPath}"`)
       console.log('opened brave-core instance (at "' + installedPath + '")')
     } else if (platformUtil.isWindows()) {
-      execSync(`"${installedPath}\\brave.exe"`)
+      exec(`"${installedPath}\\brave.exe"`)
       console.log('opened brave-core instance (at "' + installedPath + '\\brave.exe")')
     }
-    app.exit()
   } catch (e) {
     console.log('ERROR: tried to launch brave-core (at "' + installedPath +
       '"), but an exception was thrown:\n' + e.toString())