Closed
Bug 403650
Opened 17 years ago
Closed 15 years ago
expose API in nsIFaviconService to expire / delete all favicons
Categories
(Toolkit :: Places, defect)
Toolkit
Places
Tracking
()
RESOLVED
FIXED
mozilla1.9.2a1
People
(Reporter: moco, Assigned: mak)
References
Details
(Whiteboard: [fixed in bug 476636])
add a method to nsIFaviconService to expire / delete all favicons. it might be better to expire them all (change the expiration value) than to delete them. dolske needs this for a bug he's working on (see bug #389273) for existing profiles (Fx 3 beta 1 and earlier) once he lands, we'd probably to a places schema bump (but not migrate any data). instead we'd call nsIFaviconService::ExpireAllFavicons(). Then, dolske's code will replace large favicons with scaled versions. Note, dolske could also use this to properly clear his places.sqlite that has some incorrectly scaled favicons. Another reason to make this a method is to allow someone to write an extension to clear favicons. I'm not sure if fx 2 had a way to do this, but I know that I've looked up how to do it for Safari.
Comment 1•17 years ago
|
||
Another thought: Since the goal is just to help make sure that any giant icons currently stored get flushed out, we might not want to immediately expire everything (which seems like it might be annoying the next launch, when everything is fetched again)... Instead, we could adjust the expire time to a random value between now and now+5days, and allow icons to slowly be refreshed that way. Then again, these are alphas and betas so it might be easiest to just blow away the whole cache.
Assignee | ||
Comment 2•16 years ago
|
||
remove all favicons is partly implmented in bug 460300
Depends on: 460300
Assignee | ||
Updated•16 years ago
|
Product: Firefox → Toolkit
QA Contact: places → places
Assignee | ||
Updated•16 years ago
|
Whiteboard: [good first bug]
Assignee | ||
Updated•16 years ago
|
Summary: add a method to nsIFaviconService to expire / delete all favicons → expose API in nsIFaviconService to expire / delete all favicons
Assignee | ||
Comment 3•15 years ago
|
||
fixed in bug 476636 nsIFaviconsService::ExpireAllFavicons() It is an async method, notifies through observer service on completion.
Status: NEW → RESOLVED
Closed: 15 years ago
Depends on: 476636
Resolution: --- → FIXED
Whiteboard: [good first bug] → [fixed in bug 476636]
Target Milestone: --- → mozilla1.9.2a1
Assignee | ||
Updated•15 years ago
|
Assignee: nobody → mak77
You need to log in
before you can comment on or make changes to this bug.
Description
•