<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sat, Feb 6, 2016 at 10:21 AM, David Faure <span dir="ltr"><<a href="mailto:faure@kde.org" target="_blank">faure@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Unaware of this discussion (no time to read kde-devel), I have done exactly<br>
this during January. KIO (starting from 5.19) now has a KIOGui library with a<br>
FavIconRequestJob, which removes the dependency on the kded module<br>
provided by libkonq (I'll kill it soon, since it wasn't released).<br>
KIO::favIconForUrl(), which you mention in your mail, now uses the cache<br>
where FavIconRequestJob stores icons. Please port tellico to<br>
FavIconRequestJob and report any problems to me (or on the<br>
kde-frameworks-devel list, or in bugzilla). </blockquote><div><br></div><div>FavIconRequestJob works great. I do see a behavioral regression in KIconLoader::loadIcon(). In KDE4, the favicon icon name started with "favicons/" and KIconLoader would use it as an overlay for a larger pixmap. The code in KIconLoader::loadIcon() checks that the icon name starts with "favicons/". With the current state, the icon name is an absolute path so the overlay doesn't work.</div><div><br></div><div><div>> // Special case for absolute path icons.</div><div>> if (name.startsWith(QLatin1String("favicons/"))) {</div><div>>   favIconOverlay = true;</div><div>>   name = QStandardPaths::writableLocation(QStandardPaths::GenericCacheLocation) + '/' + name + ".png";</div><div>> }</div></div><div><br></div><div>Not a big deal at all and easy enough to compensate for. I'm not sure what the appropriate fix is. Perhaps changing KIconLoader to check for an absolute path that includes "favicons/" ?</div><div><br></div><div>Thanks again for FavIconRequestJob in 5.19, David.</div><div><br></div><div>Robby</div></div></div></div>