Skip to content
This repository has been archived by the owner on Nov 1, 2021. It is now read-only.

gtk-primary-selection: support multiple devices #1380

Merged

Conversation

emersion
Copy link
Member

When a client was creating multiple data devices for the same seat, we were
only creating one resource. This is a protocol error.

Instead, create one offer per data device.

This commit also makes offers inert when their source is destroyed.

Fixes part of #1041
Supersedes #1113

When a client was creating multiple data devices for the same seat, we were
only creating one resource. This is a protocol error.

Instead, create one offer per data device.

This commit also makes offers inert when their source is destroyed.

Fixes part of swaywm#1041
Supersedes swaywm#1113
@emersion emersion force-pushed the gtk-primary-selection-multiple-devices branch from bbf625a to eaafd65 Compare November 21, 2018 10:16
@baloo
Copy link

baloo commented Nov 21, 2018

I gave the patch in the PR a shot with firefox 65 (hg master) and sway (master). It doesn't crash anymore when I'm selecting text in the address bar.

Thank you!

@uwqwxing886
Copy link

uwqwxing886 commented Nov 21, 2018

After applying the patch, the ipc_channel_posix.cc errors seen in swaywm/sway#3170 and other issues no longer occur when running firefox (at least with light use).

[Child 20714, Chrome_ChildThread] WARNING: pipe error (40): Connection reset by peer: file /var/cache/pkg/firefox-wayland/src/mozilla-unified/ipc/chromium/src/chrome/common/ipc_channel_posix.cc, line 363

@emersion emersion merged commit 42e8e3e into swaywm:master Nov 21, 2018
@emersion emersion deleted the gtk-primary-selection-multiple-devices branch November 21, 2018 21:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants