DescriptionPrint too-long command line
When RegisterApplicationRestart fails with E_INVALIDARG it is supposed
to indicate that the command line is too long. This error was hit on
a recent test run and it was impossible to do further diagnosis because
the command line is not printed, even though it is printed for other
failure codes. This change prints the command line in the E_INVALIDARG
case.
The failure only happened once and was probably a Windows glitch since
all evidence is that the command line was short enough. But, still.
This change also removes some checks for win::VERSION_VISTA since Chrome
requires Windows 7 or higher, and avoids some redundant calls to
GetCommandLineString.
Review-Url: https://codereview.chromium.org/2756603002
Cr-Commit-Position: refs/heads/master@{#457297}
Committed: https://chromium.googlesource.com/chromium/src/+/fe785a58e31217e1ef0e1c8946a4e853829371f8
Patch Set 1 #
Total comments: 2
Patch Set 2 : Remove now unneeded include #
Messages
Total messages: 15 (10 generated)
|