D7092: Keep fallback icon updated
Eike Hein
noreply at phabricator.kde.org
Thu Aug 3 12:14:33 UTC 2017
hein created this revision.
Restricted Application added a project: Plasma.
REVISION SUMMARY
Windows we can't find an app icon for using the normal means
get the icon used by the windowing system in the Task Manager.
This fallback icon was then not updated when changed on the
window, only occasionally as a side-effect of cache evictions
and model data requests.
This patch notes which windows hit the fallback path, and for
those windows evicts the cache when the icon changes on the
window, causing it to be re-retrieved from the windowing system
as views respond to the dataChanged signal for the decoration
role.
Evicting the entire cache is a little bit costly, but:
- This is a fallback codepath.
- Apps conventionally update title and icon in one go, meaning the cache is often already getting evicted anyway.
- Icons don't change that often.
BUG:383017
REPOSITORY
R120 Plasma Workspace
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D7092
AFFECTED FILES
libtaskmanager/xwindowtasksmodel.cpp
To: hein, #plasma, davidedmundson
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170803/de3a9ee6/attachment.html>
More information about the Plasma-devel
mailing list