Two windows are opened when installing for Windows without admin privileges

Description

When installing Brave without admin, you’ll notice that two windows are being launched. The first window will include the brave://welcome tab and the other window will have brave://newtab opened. @srirambv @bsclifton have reproduced this several times as well.

Steps to Reproduce

1. download one of the latest stub installers for Win
2. double click on the stub installer and select “No” under the UAC prompt
3. click Yes when prompted Brave can be installed without admin privileges. Continue?

Once Brave is installed and launched, you’ll notice two windows being launched.

Actual result:

Expected result:

Should only open brave://welcome in a single window. There shouldn’t be two windows.

Reproduces how often:

100% reproducible using the above STR.

Brave version (brave://version info)

Brave	0.56.6 Chromium: 70.0.3538.67 (Official Build) beta(64-bit)
Revision	9ab0cfab84ded083718d3a4ff830726efd38869f-refs/branch-heads/3538@{#1002}
OS	Mac OS X

Reproducible on current release:

• Does it reproduce on brave-browser dev/beta builds?

Yes, currently reproducible with the dev, beta and release stub installers.

Additional Information

CCing @bsclifton @srirambv as they’ve experienced this issue as well.

[mherrmann]

Appears fixed for me as of Nightly 1.37.40.

[mherrmann]

@mihaiplesa are we passing --do-not-launch-chrome to the install action on the update server?

Edit: I was able to check this myself and it appears that no, we are not passing --do-not-launch-chrome to the install action. I think we should do so.