Add STARTF_FORCEOFFFEEDBACK to prevent busy cursor on process launch

When starting or restarting Chromium on Windows, we see the busy cursor
briefly.  This is because crashpad does not specify
STARTF_FORCEOFFFEEDBACK in the CreateProcess call.  This is similar to
other changes in Chromium for child processes to avoid the busy cursor.

Bug: 389978731
Change-Id: Ib95cd3b6ce13e3170324e63b43e46a376affc7c8
Reviewed-on: https://chromium-review.googlesource.com/c/crashpad/crashpad/+/6173930
Reviewed-by: Will Harris <wfh@google.com>
Commit-Queue: Chris Davis <chrdavis@microsoft.com>
Reviewed-by: Peter Boström <pbos@chromium.org>
Reviewed-by: Mark Mentovai <mark@chromium.org>
Reviewed-by: Will Harris <wfh@chromium.org>
This commit is contained in:
Chris Davis 2025-01-14 13:03:11 -08:00 committed by Crashpad LUCI CQ
parent a0025ad44b
commit c6d912dac1

View File

@ -438,7 +438,8 @@ bool StartHandlerProcess(
BOOL rv;
DWORD creation_flags;
STARTUPINFOEX startup_info = {};
startup_info.StartupInfo.dwFlags = STARTF_USESTDHANDLES;
startup_info.StartupInfo.dwFlags =
STARTF_USESTDHANDLES | STARTF_FORCEOFFFEEDBACK;
startup_info.StartupInfo.hStdInput = GetStdHandle(STD_INPUT_HANDLE);
startup_info.StartupInfo.hStdOutput = GetStdHandle(STD_OUTPUT_HANDLE);
startup_info.StartupInfo.hStdError = GetStdHandle(STD_ERROR_HANDLE);