758458 - Favicon's with multiple resolutions should use 32x32 version

Status: RESOLVED DUPLICATE of bug 748100

(Firefox for Android Graveyard :: General, defect)

Description

[Rehan Dalal (Inactive)]

ICO files store multiple images. Fennec should check for a 32x32 sized image in the ICO file and use it, if available. Currently it uses the 16x16 image even when a higher resolution image exists in the file.

Comment 1

[Aaron Train [:aaronmt]]

I bet this is why most Favicons look like garbage in Fennec, no?

Comment 2

[Mark Finkle (:mfinkle) (use needinfo?)]

Since we load and show favicons in Java/Android, we depend on the Android utilities to handle ICOs. THose utilities don't handle multiple bitmaps in an ICO file, so we get the first one it finds. Supporting multiple bitmaps would require new code, or loading via Gecko somehow.

Comment 3

[:Margaret Leibovic]

(In reply to Mark Finkle (:mfinkle) from comment #2)
> Supporting multiple bitmaps
> would require new code, or loading via Gecko somehow.

Gecko doesn't handle this. See bug 419588.

Comment 4

[Chris Kitching [:ckitching]]

This bug describes the same problem as Bug 748100 - the need for a Java solution to decode ICO files (With, in this case, emphasis on extracting the most resolute image therefrom). (Fixed by the same patch, as well)
I think this means close-as-duplicate time, then?