[mutter/wip/carlosg/update-pad-osd-actions: 362/362] core: Let pad mode switch events always go through MetaInputSettings

[Carlos Garnacho <carlosg src gnome org>]

commit 78f853126f0b2ce8363bf4ffb39556db202eaa47
Author: Carlos Garnacho <carlosg gnome org>
Date:   Mon Dec 16 13:53:26 2019 +0100

    core: Let pad mode switch events always go through MetaInputSettings
    
    We used to inhibit all pad actions while the OSD is shown, but one we
    would actually want to handle are mode switches while the OSD is open.
    So it has an opportunity to catch up to the mode switch.
    
    This lets MetaInputSettings reflect the mode switch (eg. when querying
    action labels), so the OSD has an opportunity to update the current
    actions.
    
    https://gitlab.gnome.org/GNOME/mutter/merge_requests/975

 src/core/events.c | 30 ++++++++++++++++++++++++------
 1 file changed, 24 insertions(+), 6 deletions(-)
---
diff --git a/src/core/events.c b/src/core/events.c
index eedb04a01..615e9aeb1 100644
--- a/src/core/events.c
+++ b/src/core/events.c
@@ -240,13 +240,31 @@ meta_display_handle_event (MetaDisplay        *display,
     }
 #endif
 
-  if (!display->current_pad_osd &&
-      (event->type == CLUTTER_PAD_BUTTON_PRESS ||
-       event->type == CLUTTER_PAD_BUTTON_RELEASE ||
-       event->type == CLUTTER_PAD_RING ||
-       event->type == CLUTTER_PAD_STRIP))
+  if (event->type == CLUTTER_PAD_BUTTON_PRESS ||
+      event->type == CLUTTER_PAD_BUTTON_RELEASE ||
+      event->type == CLUTTER_PAD_RING ||
+      event->type == CLUTTER_PAD_STRIP)
     {
-      if (meta_input_settings_handle_pad_event (meta_backend_get_input_settings (backend),
+      gboolean handle_pad_event = TRUE;
+      gboolean is_mode_switch = FALSE;
+
+      if (event->type == CLUTTER_PAD_BUTTON_PRESS ||
+          event->type == CLUTTER_PAD_BUTTON_RELEASE)
+        {
+          ClutterInputDevice *pad;
+          uint32_t button;
+
+          pad = clutter_event_get_source_device (event);
+          button = clutter_event_get_button (event);
+
+          is_mode_switch =
+            clutter_input_device_get_mode_switch_button_group (pad, button) >= 0;
+        }
+
+      handle_pad_event = !display->current_pad_osd || is_mode_switch;
+
+      if (handle_pad_event &&
+          meta_input_settings_handle_pad_event (meta_backend_get_input_settings (backend),
                                                 event))
         {
           bypass_wayland = bypass_clutter = TRUE;