667905 - Figure out why we can't update icons with moz-anno:favicon in nsNavHistoryResult

Status: RESOLVED INVALID

(Toolkit :: Places, defect)

Description

[Marco Bonardo [:mak]]

when we get a pageChanged or itemChanged notification for favicons we set mFaviconURI to the new spec.
We should instead use a moz-anno:favicon: spec, to hit the db rather than the network (even if actually we hit the network cache since the favicon is most likely there).
This doesn't seem to work though, the icon is correctly setup but the view doesn't show it.

Comment 1

[Shawn Wilsher :sdwilsh]

(In reply to comment #0)
> This doesn't seem to work though, the icon is correctly setup but the view
> doesn't show it.
Do we get a network error, or something else?  Is it possible that it has not yet been written out to disk?

Comment 2

[Marco Bonardo [:mak]]

likely superseded by work for bug 13459569