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)
committerMat <mail@mathias.is>
Sat, 12 Nov 2022 01:37:34 +0000 (03:37 +0200)
commit9b74027deaa4995ef582fdaf970ad881c3263cc1
tree58eef2281d0213e38ea0060a032394518fa926d1
parent5b98e171b25c3cf818a1eee26445fa0370449898
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.

Closes: https://gitlab.gnome.org/GNOME/gtk/-/issues/5301
gdk/wayland/gdkdevice-wayland.c