D18731: Replace KIconThemes dependency with equivalent QIcon usage

[Volker Krause]

vkrause added a comment.


  Tested Konqueror bookmarks. New bookmarks don't seem to get the favicon set until you explicitly trigger the "Update Favicon" action, afterwards they have it and it remains across restarts. So this seems to work as expected.
  Regarding the indirect dependencies, yes, this does not actually change much on its own. I run into this while debugging icon loading issues in Okular on Android, which seems related to KIconThemes interfering, so less places where I need to strip it out help.

REPOSITORY
  R294 KBookmarks

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D18731

To: vkrause, davidedmundson
Cc: broulik, cfeck, kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190209/5e9c3c51/attachment.html>