Bug 187868

Summary: [WinCairo] Add implementation for setting cursors
Product: WebKit Reporter: Stephan Szabo <stephan.szabo>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, commit-queue, ews-watchlist, Hironori.Fujii, rniwa, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
ews-watchlist: commit-queue-
Archive of layout-test-results from ews106 for mac-sierra-wk2
none
Patch reupload
none
Patch none

Description Stephan Szabo 2018-07-20 11:16:33 PDT
[WinCairo] Add implementation for setting cursors
Comment 1 Stephan Szabo 2018-07-20 11:20:14 PDT
Created attachment 345465 [details]
Patch
Comment 2 EWS Watchlist 2018-07-20 12:41:46 PDT
Comment on attachment 345465 [details]
Patch

Attachment 345465 [details] did not pass mac-wk2-ews (mac-wk2):
Output: https://webkit-queues.webkit.org/results/8601823

New failing tests:
accessibility/smart-invert-reference.html
Comment 3 EWS Watchlist 2018-07-20 12:41:48 PDT
Created attachment 345474 [details]
Archive of layout-test-results from ews106 for mac-sierra-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-sierra-wk2  Platform: Mac OS X 10.12.6
Comment 4 Stephan Szabo 2018-07-20 13:14:07 PDT
Created attachment 345477 [details]
Patch reupload
Comment 5 Fujii Hironori 2018-07-22 20:48:00 PDT
The original implementation handle WM_SETCURSOR. Do you need to do it?
https://github.com/WebKit/webkit/blob/95edf431181ad3e174b3112afed085898d8d3f95/Source/WebKit2/UIProcess/win/WebView.cpp#L211
Comment 6 Stephan Szabo 2018-07-23 13:48:43 PDT
Created attachment 345603 [details]
Patch

The WM_SETCURSOR stuff was already in WebView, so tied back into that path from the page client implementation.
Comment 7 WebKit Commit Bot 2018-07-23 18:35:54 PDT
Comment on attachment 345603 [details]
Patch

Clearing flags on attachment: 345603

Committed r234129: <https://trac.webkit.org/changeset/234129>
Comment 8 WebKit Commit Bot 2018-07-23 18:35:56 PDT
All reviewed patches have been landed.  Closing bug.
Comment 9 Radar WebKit Bug Importer 2018-07-23 18:36:22 PDT
<rdar://problem/42524657>