gdk/wayland: Fix button mask calculation on button events
authorCarlos Garnacho <carlosg@gnome.org>
Wed, 2 Nov 2022 16:11:10 +0000 (17:11 +0100)
committerSimon McVittie <smcv@debian.org>
Tue, 6 Dec 2022 11:52:29 +0000 (11:52 +0000)
commitd716e67ae0e24a2bab29f5c84868adc0d418a4e5
treeac4059588d9dd55fb8f683260c7da250071157a2
parent089f36fedd7dc6ff9d02f7d4459bc02dd56c9362
gdk/wayland: Fix button mask calculation on button events

There's 2 things broken here:
- The mask was calculated on top of the GDK button (i.e. skipping
  4-7 buttons), so GDK_BUTTON4_MASK and GDK_BUTTON5_MASK were not
  assigned. This is now calculated on the (continuous) BTN_ evcodes
  so it is guaranteed that the next 2 physical buttons (i.e.
  back/forward) get these two places in the mask assigned.
- Furthermore, these buttons would be pushed to places in the
  modifier mask that they didn't belong to. It is now checked hard
  that only the first 5 buttons enable a modifier flag.

Overall, this ensures that no event masks with bonkers values are
forwarded, and that no stale implicit grabs are left after additional
buttons are pressed.

Bug: https://gitlab.gnome.org/GNOME/gtk/-/issues/5301
Origin: upstream, 4.8.3, commit:9b74027deaa4995ef582fdaf970ad881c3263cc1

Gbp-Pq: Name gdk-wayland-Fix-button-mask-calculation-on-button-events.patch
gdk/wayland/gdkdevice-wayland.c