Differential D8082

Also send Wayland clients to a new desktop if their desktop was removed
ClosedPublic
Actions

Authored by graesslin on Oct 1 2017, 9:03 AM.

Details

Reviewers
broulik
Group Reviewers
KWin
Plasma
Commits
R108:0d96e60b79a7: Also send Wayland clients to a new desktop if their desktop was removed
Summary

So far the method only operated on X11 clients. So when the last desktop
got removed Wayland clients were still on it. As the auto test showed:
this results in a crash.

Credits go to code coverage as it showed that area as red, which made me
look on it and realize this must be broken.

Test Plan

New test case added

Diff Detail

Repository
R108 KWin
Branch
virtual-desktop-remove-send-client-5.11
Lint
No Linters Available
Unit
No Unit Test Coverage
graesslin created this revision.Oct 1 2017, 9:04 AM
Restricted Application added a project: KWin. · View Herald TranscriptOct 1 2017, 9:04 AM
Restricted Application added subscribers: kwin, plasma-devel. · View Herald Transcript
broulik accepted this revision.Oct 1 2017, 9:09 AM
This revision is now accepted and ready to land.Oct 1 2017, 9:09 AM
Closed by commit R108:0d96e60b79a7: Also send Wayland clients to a new desktop if their desktop was removed (authored by graesslin). · Explain WhyOct 1 2017, 1:26 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 20178

View Options

autotests/integration/CMakeLists.txt

Loading...
View Options

autotests/integration/virtual_desktop_test.cpp

Loading...
View Options

workspace.cpp

Loading...