Bug 230541 – [GTK] Allow sending precise mouse wheel events in LayoutTests

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Chris Lord]

Currently in TestRunner you can send a mouse wheel event with start/end phases, but you can't actually send pixel or sub-pixel precise mouse-wheel events. This is a bit of a pain for GTK specifically because kinetic scrolling can only be exercised with precise mouse-wheel events. This is actually the same for other platforms that use sync scrolling and the base scroll animation classes, but I suspect GTK is the main consumer here (and the only one I can easily test).

We may well want this behaviour on other non-Mac/iOS platforms, such as WPE, but I'm dealing with GTK first as it's the easiest to test.

[Chris Lord]

Created attachment 438799 [details]
Patch

[EWS Watchlist]

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

[Simon Fraser (smfr)]

Comment on attachment 438799 [details]
Patch

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

> Source/WebKit/UIProcess/API/gtk/WebKitWebViewBaseInternal.h:48
> +WK_EXPORT void webkitWebViewBaseSetWheelEventsArePrecise(WebKitWebViewBase*, bool);

HasPreciseDeltas

> Tools/WebKitTestRunner/EventSenderProxy.h:66
> +    void setMouseWheelIsPrecise(bool);

I think setMouseWheelHasPreciseDeltas() would be a better name.

> Tools/WebKitTestRunner/InjectedBundle/Bindings/EventSendingController.idl:41
> +    undefined setMouseWheelIsPrecise(boolean isPrecise);

setMouseWheelHasPreciseDeltas

> Tools/WebKitTestRunner/InjectedBundle/EventSendingController.cpp:382
> +    WKBundlePageForceRepaint(InjectedBundle::singleton().page()->page()); // Triggers a scrolling tree commit.

This is a weird thing to do here, and could affect test behavior.

> Tools/WebKitTestRunner/InjectedBundle/EventSendingController.h:61
> +    void setMouseWheelIsPrecise(bool);

setMouseWheelHasPreciseDeltas

[Chris Lord]

Created attachment 438936 [details]
Patch

[Chris Lord]

(In reply to Simon Fraser (smfr) from comment #3)
> Comment on attachment 438799 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=438799&action=review
> 
> > Source/WebKit/UIProcess/API/gtk/WebKitWebViewBaseInternal.h:48
> > +WK_EXPORT void webkitWebViewBaseSetWheelEventsArePrecise(WebKitWebViewBase*, bool);
> 
> HasPreciseDeltas
> 
> > Tools/WebKitTestRunner/EventSenderProxy.h:66
> > +    void setMouseWheelIsPrecise(bool);
> 
> I think setMouseWheelHasPreciseDeltas() would be a better name.
> 
> > Tools/WebKitTestRunner/InjectedBundle/Bindings/EventSendingController.idl:41
> > +    undefined setMouseWheelIsPrecise(boolean isPrecise);
> 
> setMouseWheelHasPreciseDeltas
> 
> > Tools/WebKitTestRunner/InjectedBundle/EventSendingController.cpp:382
> > +    WKBundlePageForceRepaint(InjectedBundle::singleton().page()->page()); // Triggers a scrolling tree commit.
> 
> This is a weird thing to do here, and could affect test behavior.
> 
> > Tools/WebKitTestRunner/InjectedBundle/EventSendingController.h:61
> > +    void setMouseWheelIsPrecise(bool);
> 
> setMouseWheelHasPreciseDeltas

All sounds good, I went with the shorter setWheelHasPreciseDeltas, but don't mind changing it if you don't like that.

[EWS]

Committed r282952 (242042@main): <https://commits.webkit.org/242042@main>

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

[Radar WebKit Bug Importer]

<rdar://problem/83436653>