Closed Bug 741010 Opened 9 years ago Closed 9 years ago
Don't save favicon if loaded from DB
We currently store every loaded favicon to the DB - even if that favicon just came from the DB itself. We only need to update the DB if the favicon was downloaded.
Comment on attachment 611084 [details] [diff] [review] patch Do we have any tests for favicons? If not, let's file a bug to add a few.
Attachment #611084 - Flags: review?(mark.finkle) → review+
(In reply to Mark Finkle (:mfinkle) from comment #2) > Comment on attachment 611084 [details] [diff] [review] > patch > > Do we have any tests for favicons? If not, let's file a bug to add a few. No, we don't have tests for the stuff in Favicons.java. We definitely should.
Filed bug 741657.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 14
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.