bruns created this revision. bruns added reviewers: Plasma, davidedmundson. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision.
REVISION SUMMARY QDir.entryInfoList(NoDotAndDotDot) always returns an empty list, as it requires one of QDir::Files, QDir::Dirs or QDir::Drives or any combination thereof. Instead of implementing it manually, use QDir::removeRecursively(). TEST PLAN Watch ~/.cache/krunner/ When krunner is closed, KRunner-Favicons-firefox-default should be emptied and deleted. REPOSITORY R120 Plasma Workspace BRANCH favicon_dir_cleanup REVISION DETAIL https://phabricator.kde.org/D15391 AFFECTED FILES runners/bookmarks/faviconfromblob.cpp To: bruns, #plasma, davidedmundson Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart