[webkit-dev] Async IconDatabase and reading icons from disk on
startup
Adam Treat
treat at kde.org
Sun May 11 22:10:15 PDT 2008
On Sunday 11 May 2008, Adam Treat wrote:
> On Sunday 11 May 2008, Darin Adler wrote:
> > Or you could do as Safari does and call iconForPageURL twice; the
> > first time it's "display the icon if we already have it" but the
> > second time it's "I heard you have a new icon, lets display the new
> > icon we now know about".
> >
> > It would probably be cleaner if triggering the load of the icon was
> > completely separate from actually fetching the icon data.
>
> Right. I would suggest that iconForPageURL should always return 0 unless
> and until readIconForPageURLFromDisk is called which actually triggers the
> load.
>
> It should be noted that *nothing* currently uses
> readIconForPageURLFromDisk() ;)
>
> I guess I just couldn't believe that Safari calls iconForPageURL twice. It
> is very unclear API.
Simon,
I don't think QtWebKit API deals with this asynchronous nature of the
IconDatabase at all. I'm not sure how to deal with it as it would seem that
QWebSettings::iconForUrl() needs to be deprecated and possibly all the icon
database API in QWebSettings.
Looks like we need at a minimum a public API signal that the icon has been
loaded (either from disk or network) and that will require a new class as
QWebSettings is not a QObject. QWebIconDatabase ?? :(
Cheers,
Adam
More information about the webkit-dev
mailing list