<table><tr><td style="">hein added a comment.
</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/D7092" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>I'm possibly wrong, but instead of the new set couldn't you<br />
 if (appDataCache.contains(window) && appDataCache[window].icon.isValid())</p></blockquote>

<p>I don't understand on multiple levels :)</p>

<p>There's no QIcon::isValid, and even if there was, the icon being in the cache and being valid could be the result of the the preferred codepath (from .desktop file) in which case we don't want to emit DecorationRole based on a NET::WMIcon prop change. Basically this patch is about "react to NET::WMIcon for windows we actually show it for, but only those".</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/D7092" rel="noreferrer">https://phabricator.kde.org/D7092</a></div></div><br /><div><strong>To: </strong>hein, Plasma, davidedmundson<br /><strong>Cc: </strong>plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas<br /></div>