<div>graesslin created this revision.<br />
graesslin added reviewers: Plasma, hein.<br />
Restricted Application added a project: Plasma.<br />
Restricted Application added a subscriber: plasma-devel.</div><br /><div><strong>REVISION SUMMARY</strong><div><p>So far when the PlasmaWindowManagement went down the PlasmaWindowModel<br />
deleted all windows. This is not needed as the PlasmaWindow has the<br />
PlasmaWindowManagement as parent, so it will be destroyed anyway.</p>

<p>The delete in PlasmaWindowModel risks that other signals connecting<br />
to the same signal will operate on deleted data, which they did not<br />
expect. So it's better to not delete here and let the later auto<br />
delete handle it.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rKWAYLAND KWayland</div></div></div><br /><div><strong>BRANCH</strong><div><div>dont-delete-window-from-model</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D1623" rel="noreferrer">https://phabricator.kde.org/D1623</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/client/plasmawindowmodel.cpp</div></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>graesslin, Plasma, hein<br /><strong>Cc: </strong>plasma-devel, sebas<br /></div>