[mutter/wip/carlosg/frames-client: 994/1005] x11: Drop unnecessary check to ignore crossing events




commit 3f489d1805313fce4689834bb78723e229c71afe
Author: Carlos Garnacho <carlosg gnome org>
Date:   Wed Sep 7 16:53:44 2022 +0200

    x11: Drop unnecessary check to ignore crossing events
    
    This check dates all the way back to commit ac2aa5337dac. At the time, the
    window switcher was an actual X window, that could generate crossing events
    if popped up under the pointer. Checking for this kind of crossing events
    made sense back at the time in order not to break focus-follows-mouse as
    it's been behaving for long.
    
    But now, this UI is all Clutter widgetry, which in the worst case (X11
    sessions, of course) it will update the stage window shape to make these
    parts clickable. This happens in other places of code that do already
    check for ignoring crossing events.
    
    Underneath, this looked up for a Mutter-local GdkWindow of type
    GDK_WINDOW_TEMP, only the main MetaFrames window matches those characteristics
    nowadays, notably no window switcher popups. Since the remaining window is
    never unmapped (until perhaps shutdown), the paths were functionally dead.

 src/x11/events.c | 12 ------------
 1 file changed, 12 deletions(-)
---
diff --git a/src/x11/events.c b/src/x11/events.c
index 03c3942d84..8bacd0d2ab 100644
--- a/src/x11/events.c
+++ b/src/x11/events.c
@@ -1924,18 +1924,6 @@ meta_x11_display_handle_xevent (MetaX11Display *x11_display,
 
   input_event = get_input_event (x11_display, event);
 
-  if (event->type == UnmapNotify)
-    {
-      if (meta_ui_window_should_not_cause_focus (x11_display->xdisplay,
-                                                 modified))
-        {
-          meta_display_add_ignored_crossing_serial (display, event->xany.serial);
-          meta_topic (META_DEBUG_FOCUS,
-                      "Adding EnterNotify serial %lu to ignored focus serials",
-                      event->xany.serial);
-        }
-    }
-
   if (handle_input_xevent (x11_display, input_event, event->xany.serial))
     {
       bypass_gtk = bypass_compositor = TRUE;


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]