<table><tr><td style="">broulik created this revision.<br />broulik added reviewers: Plasma, hein.<br />Restricted Application added a project: Plasma.<br />Restricted Application added a subscriber: plasma-devel.<br />broulik requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D11096">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>The window name and user visible name are not used in the app identification heuristic, so a change here cannot result in the icon, app id, app name, generic name, PID or launcher changing.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Changed files in Qt Creator, noticed that it also erroneously signals a geometry change, when I ignored that, I verified that it was the name change code path updating the label and not the geometry one (which also nukes app info cache)</p>

<p>Also verified that the icon isn't always signalled to have changed just because the window name changed (can be expensive for apps where we don't know the service and have to query the window icon)</p>

<p>Inspected TaskTools namespace and didn't find any use of KWindowInfo (and as such no use of window name/visible name)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R120 Plasma Workspace</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11096">https://phabricator.kde.org/D11096</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>libtaskmanager/xwindowtasksmodel.cpp</div></div></div><br /><div><strong>To: </strong>broulik, Plasma, hein<br /><strong>Cc: </strong>plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>