Differential D1261

[server] Don't emit unmapped if the Surface wasn't mapped
ClosedPublic
Actions

Authored by graesslin on Mar 30 2016, 8:18 AM.

Details

Reviewers
sebas
Group Reviewers
Plasma
Commits
R127:62a43f0c0cc8: [server] Don't emit unmapped if the Surface wasn't mapped
Summary

If a Surface doesn't have a buffer attached and a null buffer gets
attached the buffer state doesn't really change. Thus neither the
unmapped signal nor the damaged signal should not be emitted.

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 3022.Mar 30 2016, 8:18 AM
graesslin retitled this revision from to [server] Don't emit unmapped if the Surface wasn't mapped.
graesslin updated this object.
graesslin edited the test plan for this revision. (Show Details)
graesslin added a reviewer: Plasma.
Restricted Application added a project: Plasma. · View Herald TranscriptMar 30 2016, 8:18 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
sebas accepted this revision.Apr 3 2016, 12:19 PM
sebas added a reviewer: sebas.
This revision is now accepted and ready to land.Apr 3 2016, 12:19 PM
Closed by commit R127:62a43f0c0cc8: [server] Don't emit unmapped if the Surface wasn't mapped (authored by graesslin). · Explain WhyApr 4 2016, 7:24 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 3117

View Options

autotests/client/test_wayland_surface.cpp

Loading...
View Options

src/server/surface_interface.cpp

Loading...