RFR: 8320965: Scrolling on a touch enabled display fails on Wayland [v3]

Kevin Rushforth kcr at openjdk.org
Mon Jan 22 19:29:44 UTC 2024


On Mon, 18 Dec 2023 11:19:18 GMT, Jose Pereda <jpereda at openjdk.org> wrote:

>> This PR replaces the deprecated `gdk_pointer_grab` with `gdk_seat_grab`, and `gdk_pointer_ungrab ` with `gdk_seat_ungrab`, using runtime checks and wrapped functions for GTK 3.20+ (so systems without it still run with GTK 3.8+), and fixes the dragging issue on Wayland.
>
> Jose Pereda has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Add compile-time checks to GdkSeat

This dropped off my radar.

The only pending issue is the test failure I noted in an earlier comment:

> I then did a full test run on our headful test systems, and there is one new test failure -- it seems to be intermittent, although fails pretty consistently on our Ubuntu 22.04 and Ubuntu 20.04 test systems. I can reproduce it locally on a VM running Ubuntu 22.04, where it fails about 1/2 the time with this patch applied (it fails more often on our physical test systems).
> 
> ```
> DatePickerTest > testDatePickerSceneChange FAILED
>     java.lang.AssertionError: Timeout: Failed to receive onAction call.
>         at org.junit.Assert.fail(Assert.java:89)
>         at org.junit.Assert.assertTrue(Assert.java:42)
>         at test.util.Util.waitForLatch(Util.java:400)
>         at test.robot.javafx.scene.DatePickerTest.clickDatePickerCalendarPopup(DatePickerTest.java:90)
>         at test.robot.javafx.scene.DatePickerTest.testDatePickerSceneChange(DatePickerTest.java:123)
> ```
> 
> Not sure what to make of this. I am not aware of any problems with this test, but it's possible that your fix has exposed a latent issue either in the test or somewhere else.

@jperedadnr Can you try it on your system and see if you can reproduce this failure?

-------------

PR Comment: https://git.openjdk.org/jfx/pull/1305#issuecomment-1904662936


More information about the openjfx-dev mailing list