From 0ab8b17433508ae32174766a5163cdab48c22262 Mon Sep 17 00:00:00 2001 From: Carlos Garnacho Date: Wed, 8 Apr 2020 13:37:19 +0200 Subject: [PATCH] x11: Forward current selection state when initializing X11 selections Most visible with xwayland-on-demand, at the time of setting things up for X11 selections, we don't forward the current state. This makes the first started X11 app oblivious to eg. the current clipboard. Syncing selections up at the time of initializing the X11 selection stuff ensures that doesn't happen. https://gitlab.gnome.org/GNOME/mutter/-/merge_requests/1186 (cherry picked from commit 167fd07e010e25feb7ff195706bf91d7c70d1273) --- src/x11/meta-x11-selection.c | 25 ++++++++++++++++--------- 1 file changed, 16 insertions(+), 9 deletions(-) diff --git a/src/x11/meta-x11-selection.c b/src/x11/meta-x11-selection.c index 6efffda2c..03d3c3b2d 100644 --- a/src/x11/meta-x11-selection.c +++ b/src/x11/meta-x11-selection.c @@ -23,6 +23,7 @@ #include +#include "core/meta-selection-private.h" #include "x11/meta-selection-source-x11-private.h" #include "x11/meta-x11-selection-output-stream-private.h" #include "x11/meta-x11-selection-private.h" @@ -373,10 +374,9 @@ meta_x11_selection_handle_event (MetaX11Display *x11_display, } static void -owner_changed_cb (MetaSelection *selection, - MetaSelectionType selection_type, - MetaSelectionSource *new_owner, - MetaX11Display *x11_display) +notify_selection_owner (MetaX11Display *x11_display, + MetaSelectionType selection_type, + MetaSelectionSource *new_owner) { Display *xdisplay = x11_display->xdisplay; @@ -403,6 +403,7 @@ meta_x11_selection_init (MetaX11Display *x11_display) { XSetWindowAttributes attributes = { 0 }; MetaDisplay *display = meta_get_display (); + MetaSelection *selection; guint mask, i; attributes.event_mask = PropertyChangeMask | SubstructureNotifyMask; @@ -423,18 +424,24 @@ meta_x11_selection_init (MetaX11Display *x11_display) XFixesSelectionWindowDestroyNotifyMask | XFixesSelectionClientCloseNotifyMask; + selection = meta_display_get_selection (display); + for (i = 0; i < META_N_SELECTION_TYPES; i++) { + MetaSelectionSource *owner; + XFixesSelectSelectionInput (x11_display->xdisplay, x11_display->selection.xwindow, selection_to_atom (i, x11_display->xdisplay), mask); + owner = meta_selection_get_current_owner (selection, i); + notify_selection_owner (x11_display, i, owner); } - g_signal_connect (meta_display_get_selection (display), - "owner-changed", - G_CALLBACK (owner_changed_cb), - x11_display); + g_signal_connect_swapped (selection, + "owner-changed", + G_CALLBACK (notify_selection_owner), + x11_display); } void @@ -443,7 +450,7 @@ meta_x11_selection_shutdown (MetaX11Display *x11_display) MetaDisplay *display = meta_get_display (); g_signal_handlers_disconnect_by_func (meta_display_get_selection (display), - owner_changed_cb, + notify_selection_owner, x11_display); if (x11_display->selection.xwindow != None) {