NEW229557
[GTK] EventSenderProxyGtk does not implement touch event synthesis 
https://bugs.webkit.org/show_bug.cgi?id=229557
Summary [GTK] EventSenderProxyGtk does not implement touch event synthesis
Chris Lord
Reported 2021-08-26 08:13:10 PDT
As the bug title says. This and the combination of the wheel-event synthesis API not letting you specify if precise scroll events are present makes it impossible to test the behaviour of precise (e.g. touch/touchpad) scrolling on GTK. While the easiest fix for that would be to add an extra prototype for specifying hasPreciseScrollDeltas to the wheel-event synthesis API, I think implementing touch event synthesis is the nicer and more flexible way of doing this, given that API already exists, is implemented on basically every other platform and is currently just stubbed.
Attachments
Patch (17.54 KB, patch)
2021-09-06 09:31 PDT, Chris Lord 		no flags
DetailsFormatted DiffDiff
Patch (26.35 KB, patch)
2021-09-07 06:38 PDT, Chris Lord 		clord: review?
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Chris Lord
Comment 1 2021-09-06 09:31:22 PDT
Created attachment 437421 [details] Patch
Chris Lord
Comment 2 2021-09-06 09:33:45 PDT
I've not re-baselined/enabled tests in this patch yet, I'm hoping EWS makes it easier for me to determine the list of tests that will need changing/re-enabling.
EWS Watchlist
Comment 3 2021-09-06 16:26:50 PDT
Thanks for the patch. If this patch contains new public API please make sure it follows the guidelines for new WebKit2 GTK+ API. See https://trac.webkit.org/wiki/WebKitGTK/AddingNewWebKit2API
Chris Lord
Comment 4 2021-09-07 06:38:07 PDT
Created attachment 437495 [details] Patch
Carlos Garcia Campos
Comment 5 2021-09-13 00:57:55 PDT
Comment on attachment 437495 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=437495&action=review > LayoutTests/platform/gtk/TestExpectations:-590 > -# These tests require touch support > -fast/events/touch [ Skip ] > -imported/w3c/web-platform-tests/touch-events [ Skip ] I'm surprised we can unskip these, we enable touch support at runtime depending on the hardware, or did that change?
Note You need to log in before you can comment on or make changes to this bug.