Bug 229927

Summary: [Win] TestWebKitAPI.WebKit.DidNotHandleKeyDown is failing
Product: WebKit Reporter: Fujii Hironori <Hironori.Fujii>
Component: Tools / TestsAssignee: Fujii Hironori <Hironori.Fujii>
Status: RESOLVED FIXED    
Severity: Normal CC: annulen, don.olmstead, ews-watchlist, gyuyoung.kim, ross.kirsling, ryuan.choi, sergio, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch for landing none

Description Fujii Hironori 2021-09-05 18:06:50 PDT
[Win] TestWebKitAPI.WebKit.DidNotHandleKeyDown is failing

TestWebKitAPI::Util::isKeyDown is not implemented
Comment 1 Fujii Hironori 2021-09-05 18:09:16 PDT
Created attachment 437370 [details]
Patch
Comment 2 Fujii Hironori 2021-09-06 21:07:39 PDT
Created attachment 437449 [details]
Patch
Comment 3 Don Olmstead 2021-09-07 18:20:03 PDT
Comment on attachment 437449 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=437449&action=review

r=me

I think this will break the PlayStation without the implementation being there as well.

> Tools/TestWebKitAPI/Tests/WebKit/DidNotHandleKeyDown.cpp:29
> +#if !PLATFORM(GTK) && !PLATFORM(WPE)

This should probably be

#if PLATFORM(COCOA) || PLATFORM(WIN)

> Tools/TestWebKitAPI/Tests/WebKit/DidNotHandleKeyDown.cpp:72
> +#endif // !PLATFORM(GTK) && !PLATFORM(WPE)

Same
Comment 4 Fujii Hironori 2021-09-07 19:18:32 PDT
Created attachment 437579 [details]
Patch for landing
Comment 5 Fujii Hironori 2021-09-07 21:57:53 PDT
Comment on attachment 437579 [details]
Patch for landing

Clearing flags on attachment: 437579

Committed r282128 (241425@main): <https://commits.webkit.org/241425@main>
Comment 6 Fujii Hironori 2021-09-07 21:57:57 PDT
All reviewed patches have been landed.  Closing bug.
Comment 7 Radar WebKit Bug Importer 2021-09-07 21:58:22 PDT
<rdar://problem/82855497>