Bug 91944 - wl_data_source should have an event to indicate completion of a drag session
Summary: wl_data_source should have an event to indicate completion of a drag session
Status: RESOLVED FIXED
Alias: None
Product: Wayland
Classification: Unclassified
Component: wayland (show other bugs)
Version: 1.5.0
Hardware: x86-64 (AMD64) All
: medium normal
Assignee: Wayland bug list
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 91943 91948
  Show dependency treegraph
 
Reported: 2015-09-09 20:03 UTC by Michael Catanzaro
Modified: 2016-03-17 01:56 UTC (History)
3 users (show)

See Also:
i915 platform:
i915 features:


Attachments

Description Michael Catanzaro 2015-09-09 20:03:39 UTC
When GTK+ is a drag source, it assumes that a drag session has ended after it handles the data_source.send event. After handling data_source.send, it frees memory associated with the drag session and releases its manual pointer grab, causing data_offer.leave to be immediately sent to the destination client. This causes several incompatibilities with the reference weston-dnd client:

* Unlike weston-dnd, it's impossible to send multiple different types of data in the same session.
* This breaks Wayland clients that request transfer of data before the drop is performed, which is well-supported by weston-dnd.

But GTK+ does this because anything else would be wrong: there is no other way for the drag source to know when the drag session is over, other than to assume that it ends during data_source.send. The weston-dnd client does not have these issues because it has no need for a client-side grab (which must be released when the DnD session is complete) and it simply leaks all memory associated with the DnD sessions (bug #91943), which would be unacceptable for GTK+.

One way to fix this would be to add a new event, wl_data_source.drag_finished, which the compositor would send to the source client at the same time it sends wl_data_device.leave to the destination client. This has already been implemented by Carlos Garnacho in [1] and [2], though it is just a subset of the functionality added in those patches. GTK+ would then be modified to release its grab in response to that event instead.

[1] http://lists.freedesktop.org/archives/wayland-devel/2015-June/022517.html
[2] http://lists.freedesktop.org/archives/wayland-devel/2015-June/022518.html
Comment 1 Michael Catanzaro 2016-03-17 01:56:53 UTC
We now have wl_data_offer::finish


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.