Don't save favicon if loaded from DB

RESOLVED FIXED in Firefox 14

Status

()

Firefox for Android
General
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: bnicholson, Assigned: bnicholson)

Tracking

unspecified
Firefox 14
ARM
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
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.
(Assignee)

Comment 1

6 years ago
Created attachment 611084 [details] [diff] [review]
patch
Attachment #611084 - Flags: review?(mark.finkle)
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.
(Assignee)

Comment 4

6 years ago
Filed bug 741657.
(Assignee)

Comment 5

6 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/5f02ed277424
https://hg.mozilla.org/mozilla-central/rev/5f02ed277424
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 14
You need to log in before you can comment on or make changes to this bug.