From f0be763ad792150eb364cfba0a13eae3e32d1f78 Mon Sep 17 00:00:00 2001 From: Brian Clifton Date: Thu, 6 Dec 2018 11:45:31 -0700 Subject: [PATCH] Remove default browser check (and set on launch). I believe this was causing a crash during end-to-end testing --- app/darwinInit.js | 15 --------------- 1 file changed, 15 deletions(-) diff --git a/app/darwinInit.js b/app/darwinInit.js index b48313ab39c..9bc93f16fb8 100644 --- a/app/darwinInit.js +++ b/app/darwinInit.js @@ -55,15 +55,6 @@ if (process.platform === 'darwin') { return false } - const isDefaultBrowser = () => { - try { - const defaultProtocols = ['http', 'https'] - return defaultProtocols.every(p => app.isDefaultProtocolClient(p)) - } catch (e) { - return false - } - } - const installBraveCore = () => { // get path to the bundled brave-core binary const installerPath = getBraveCoreInstallerPath() @@ -111,12 +102,6 @@ if (process.platform === 'darwin') { // launch into freshly installed brave-core and append argument expected in: // https://github.com/brave/brave-browser/issues/1545 let openCmd = `open -a "${installedPath}/${appName}/" --args --upgrade-from-muon` - if (isDefaultBrowser()) { - // if user has the Muon version of Brave as their default browser, let's - // ask the user to make Brave Core their new default. Passing this extra - // argument will prompt the user to ensure they agree with the choice - openCmd += ' --make-default-browser' - } console.log('Launching brave-core') execSync(openCmd) } catch (e) {