Differential D2035

Properly handle release of a touch resource
ClosedPublic
Actions

Authored by graesslin on Jun 28 2016, 2:17 PM.

Details

Reviewers
sebas
Group Reviewers
Plasma on Wayland
Commits
R127:5c9a63b17705: Properly handle release of a touch resource
Summary

On client side use wl_touch_release to get into the proper destroy
handler on server side. There the shared destroy implementation is
used.

The test case is extended to verify the condition and ensure that
our code doesn't crash in case SeatInterface calls into the already
unbound TouchInterface.

Diff Detail

Repository
R127 KWayland
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
graesslin updated this revision to Diff 4802.Jun 28 2016, 2:17 PM
graesslin retitled this revision from to Properly handle release of a touch resource.
graesslin updated this object.
graesslin edited the test plan for this revision. (Show Details)
graesslin added a reviewer: Plasma on Wayland.
Restricted Application added a project: Plasma on Wayland. · View Herald TranscriptJun 28 2016, 2:17 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
graesslin updated this revision to Diff 4804.Jun 28 2016, 3:07 PM

Extended the test case

sebas accepted this revision.Jun 28 2016, 3:09 PM
sebas added a reviewer: sebas.
This revision is now accepted and ready to land.Jun 28 2016, 3:09 PM
Closed by commit R127:5c9a63b17705: Properly handle release of a touch resource (authored by graesslin). · Explain WhyJun 28 2016, 3:15 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 4805

View Options

autotests/client/test_wayland_seat.cpp

Loading...
View Options

src/client/touch.cpp

Loading...
View Options

src/server/touch_interface.cpp

Loading...