Bug 187862

Summary: [WinCairo] Fix detection of held modifier keys for key events
Product: WebKit Reporter: Stephan Szabo <stephan.szabo>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, commit-queue, ews-watchlist, Hironori.Fujii, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Use requested key in IsKeyInDownState
none
Archive of layout-test-results from ews206 for win-future none

Description Stephan Szabo 2018-07-20 10:29:28 PDT
[WinCairo] Fix detection of held modifier keys for key events
Comment 1 Stephan Szabo 2018-07-20 10:47:22 PDT
We were always checking VK_MENU when checking if IsKeyInDownState rather than the requested modifier.
Comment 2 Stephan Szabo 2018-07-20 10:47:55 PDT
Created attachment 345460 [details]
Use requested key in IsKeyInDownState
Comment 3 EWS Watchlist 2018-07-21 02:24:01 PDT
Comment on attachment 345460 [details]
Use requested key in IsKeyInDownState

Attachment 345460 [details] did not pass win-ews (win):
Output: https://webkit-queues.webkit.org/results/8607279

New failing tests:
http/tests/security/canvas-remote-read-remote-video-localhost.html
Comment 4 EWS Watchlist 2018-07-21 02:24:12 PDT
Created attachment 345508 [details]
Archive of layout-test-results from ews206 for win-future

The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews206  Port: win-future  Platform: CYGWIN_NT-6.1-2.9.0-0.318-5-3-x86_64-64bit
Comment 5 Stephan Szabo 2018-07-23 10:25:53 PDT
Win bot appears to be failing both with and without the patch with similar results.
Comment 6 WebKit Commit Bot 2018-07-23 10:52:39 PDT
Comment on attachment 345460 [details]
Use requested key in IsKeyInDownState

Clearing flags on attachment: 345460

Committed r234103: <https://trac.webkit.org/changeset/234103>
Comment 7 WebKit Commit Bot 2018-07-23 10:52:40 PDT
All reviewed patches have been landed.  Closing bug.
Comment 8 Radar WebKit Bug Importer 2018-07-23 10:54:11 PDT
<rdar://problem/42506025>