Closed Bug 914952 Opened 11 years ago Closed 4 years ago

Correctly decode <link> tags containing multiple sizes and handle multiple <link> tags

Categories

(Firefox for Android Graveyard :: Theme and Visual Design, defect)

ARM
Android
defect
Not set
minor

Tracking

(Not tracked)

RESOLVED INCOMPLETE

People

(Reporter: ckitching, Unassigned)

References

Details

A <link> tag, such as one used to specify a Favicon, may specify multiple sizes. For example, if the payload is a container format such as ICO or ICNS, then the tag should (But usually doesn't) list all the different sizes of icon available. Alternatively, the sizes can be used to specify sizes to which it is valid to scale an otherwise perfectly scalable icon. For example, given an SVG payload (See also: Bug 914025), the website author my stipulate via the sizes attribute that it shall only be scaled to certain sizes (And should not be arbitrarily upscaled). They may then elect to specify an alternative favicon for larger sizes. Currently, the policy of the Favicon system is to grab the largest, shiniest one we can, and fall back repeatedly on lesser-attractive icons until we either succeed in decoding one or run out of things to try. Continuing with this, when reading <link> tags with multiple sizes we need to parse the complete list to determine which size is the largest available.
Assignee: nobody → chriskitching
Assignee: chriskitching → nobody
Merging the slight extra twiddle that Bug 1070252 pointed out, since the ensuing patch should straightforwardly resolve both problems (they all just need to feed into the same optimal favicon selection scheme).
Summary: Correctly decode <link> tags containing multiple sizes. → Correctly decode <link> tags containing multiple sizes and handle multiple <link> tags
We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → INCOMPLETE
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.