Bug 249120

Summary: [Win] REGRESSION(257142@main): debug logs aren't output to the debug console because HAVE_ISDEBUGGERPRESENT macro isn't defined
Product: WebKit Reporter: Fujii Hironori <Hironori.Fujii>
Component: CMakeAssignee: Fujii Hironori <Hironori.Fujii>
Status: RESOLVED FIXED    
Severity: Normal CC: annulen, benjamin, cdumez, cmarcelo, don.olmstead, ews-watchlist, gyuyoung.kim, ryuan.choi, sergio
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=247927
Attachments:
Description Flags
Patch none

Description Fujii Hironori 2022-12-11 21:28:17 PST
Since 257142@main (bug#247927)
Comment 1 Fujii Hironori 2022-12-11 21:30:25 PST
> check_symbol_exists(IsDebuggerPresent debugapi.h var1)
This doesn't find it.

> check_symbol_exists(IsDebuggerPresent windows.h var1)
This finds it.

In debugapi.h, IsDebuggerPresent is guarded by a condition:

> #if (_WIN32_WINNT >= 0x0400) || (_WIN32_WINDOWS > 0x0400)
> WINBASEAPI
> BOOL
> WINAPI
> IsDebuggerPresent(
>     VOID
>     );
> 
> #endif
Comment 2 Fujii Hironori 2022-12-11 21:35:26 PST
I don't know why a Windows API IsDebuggerPresent has to be checked in CMake time. 
180570@main (bug#162368) added the check.
Comment 3 Fujii Hironori 2022-12-11 22:20:27 PST
Created attachment 464001 [details]
Patch
Comment 4 EWS 2022-12-12 12:29:56 PST
Committed 257751@main (cce0a08316e8): <https://commits.webkit.org/257751@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 464001 [details].