Skip to content
This repository has been archived by the owner on Jan 19, 2022. It is now read-only.

Use startHidden and noShell attribute to hide command prompt window #41

Merged
merged 1 commit into from
May 22, 2018

Conversation

makotokato
Copy link
Member

Actually, when using adbhelper on Windows, command prompt is showen at startup. Since Firefox 55 and 58 have additional options, I would like to use it to hide command prompt window on Windows.

@jryans
Copy link
Contributor

jryans commented May 22, 2018

Thanks, this seems reasonable to me! (Sorry for the delay, I only just noticed this while digging through my inbox... Feel free to request a review in the future!)

There are a few other places where we use Subprocess instead, which doesn't appear to offer these flags at the moment. If you hit those cases on Windows, perhaps we should consider changing to an approach that allows hiding the command prompt there as well.

@jryans
Copy link
Contributor

jryans commented May 22, 2018

I have asked Patrick to add you to the DevTools GitHub team so that you can merge things when you're ready to do so. For now, I'll merge this change. Thanks again!

@jryans jryans merged commit 5d171a6 into mozilla:master May 22, 2018
@makotokato makotokato deleted the hide-cmd branch August 2, 2018 06:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants