Hi, On Mon, Jan 14, 2019 at 10:15:04PM +0100, Jakub Janku wrote: > > The comment refers to the gtk_clipboard_clear() not > > owner-changed event so you are right, of course. This comment > > was probably a left over from different interactions that I > > did in this code :( > > > > I'll send a fix for the documentation. > > > > For the other changes, I'd love to hear your opinion too. Did > > not get 2/3 and 3/3 patches acked but more keen to discuss > > the idea. > > > > I was wrong in regards to clipboard managers on Wayland, that > > is, clipboard managers on Wayland *can* get clipboard data of > > spice-gtk while spice-gtk's widgets are holding the focus :( > > That's weird. I didn't think this was possible. > I've just had a very brief look at the code of GPaste: > > https://github.com/Keruspe/GPaste/blob/master/src/daemon/gpaste-daemon.c > > the main function includes the following lines: > | /* FIXME: remove this once gtk supports clipboard correctly > on wayland */ > | gdk_set_allowed_backends ("x11"); > This probably indicates that the clipboard manager actually runs on > XWayland, which would explain why it's able to eavesdrop on the > clipboard without having focus. > It is the same "workaround" that we use in vdagent.
Ah, good. Interesting that remote-viewer running on Wayland could
have its clipboard data fetched by another application running
with x11 backend.
> > I didn't check how that happen but I'd rather avoid/deny that to
> > happen..
> >
> > Thanks again Jakub,
> >
> > > > + if (gtk_clipboard_get_owner(clipboard) == G_OBJECT(self)) {
> > > > + return;
> > > > + }
> > > > +
> > > > + s->clipboard_by_guest[selection] = FALSE;
> > > > + s->clip_hasdata[selection] = TRUE;
> > > > + if (s->auto_clipboard_enable && !read_only(self))
> > > > + gtk_clipboard_request_targets(clipboard, clipboard_get_targets,
> > > > + get_weak_ref(self));
> > > > }
> > > >
> > > > typedef struct
> > > > --
> > > > 2.20.1
> > > >
> > >
> > > Cheers,
> > > Jakub
> >
> >
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/spice-devel
