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
Branch
no-unmapped-for-not-mapped-surface
Lint
No Linters Available
Unit
No Unit Test Coverage
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 3022

View Options

autotests/client/test_wayland_surface.cpp

Loading...
View Options

src/server/surface_interface.cpp

Loading...