Skip dependent tests until that.
Skipped a test in http://trac.webkit.org/changeset/96699
Created attachment 146019 [details] Patch
Comment on attachment 146019 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=146019&action=review Generally looks good, but there are some things to complain about. > Tools/WebKitTestRunner/InjectedBundle/Bindings/EventSendingController.idl:33 > + void continuousMouseScrollBy (in long x, in long y, in [Optional] boolean paged); Extra space after function name. > Tools/WebKitTestRunner/gtk/EventSenderProxyGtk.cpp:395 > + gdk_event_set_device(event, gdk_device_manager_get_client_pointer(gdk_display_get_device_manager(gdk_window_get_display(event->scroll.window)))); > + > + // For more than one tick in a scroll, we need smooth scroll event > +#if GTK_CHECK_VERSION(3, 3, 18) > + if ((horizontal && vertical) || horizontal > 1 || horizontal < -1 || vertical > 1 || vertical < -1) { > + event->scroll.direction = GDK_SCROLL_SMOOTH; > + event->scroll.delta_x = -horizontal; > + event->scroll.delta_y = -vertical; > + > + sendOrQueueEvent(event); > + return; > + } > +#endif Why did you have to change on this function? It was already there. Please describe it in the changelog. > Tools/WebKitTestRunner/mac/EventSenderProxy.mm:401 > +void EventSenderProxy::continuousScrollBy(int x, int y, bool paged) > +{ Why is it not continuousMouseScrollBy? Have you build it on Mac?
Fix for #88070 already includes the changes for WebKitTestRunner.
Comment on attachment 146019 [details] Patch Cleared review? from attachment 146019 [details] so that this bug does not appear in http://webkit.org/pending-review. If you would like this patch reviewed, please attach it to a new bug (or re-open this bug before marking it for review again).