218651 - Favicon isn't appearing

Status: VERIFIED FIXED

(Core :: Graphics: ImageLib, defect)

Description

[dobz]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.5a) Gecko/20030824 Mozilla Firebird/0.6.1
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.5a) Gecko/20030824 Mozilla Firebird/0.6.1

Favicon on this webpage work with Internet Explorer and Konqueror, but not with
Mozilla or Mozilla Firebird

HTML is (I think) correct
link REL="shortcut icon" HREF="/favicon/favicon.ico" TYPE="image/x-icon"

And so is the icon file since other browsers can display it

Reproducible: Always

Steps to Reproduce:
1. Go to http://abusenospam.free.fr/favicon/test.html
2. See no favicon
3.

Actual Results:  
Favicon is not displayed

Expected Results:  
Mozilla should have displayed it like konqueror / ie do

Comment 1

[Stephen Donner [:stephend] Not actively reading bugmail]

Try it without the 'shortcut', that might work.

For example:

<link rel="icon" href="http://mozilla.org/images/mozilla-16.png" type="image/png">

Comment 2

[tor]

Reassign to the bmp/ico expert.

Comment 3

[tor]

Reporter: what program was used to generate this ICO?  It has a reserved field
in the image format set to a value we didn't expect.

Comment 4

[tor]

Attachment: ignore contents of directory entry reserved field

Comment 5

[Christian :Biesinger (don't email me, ping me on IRC)]

-> tor, as he has a patch

Comment 6

[dobz]

Sorry, I don't know what program was used to generate the icon.

Actually, this is the favicon of a site that I visit regulary, and I was annoyed
that the favicon would work in other browsers but not in Mozilla.

I only put it on a separate & trvial web page for convenience.

I could try asking the webmaster of the site if you feel this is important, but
he probably did it in 10 minutes a couple of years ago with some shareware, so
I'm not sure he'd remember the name

Comment 7

[tor]

Checked in.

Comment 8

[Stephen Donner [:stephend] Not actively reading bugmail]

http://abusenospam.free.fr/favicon/test.html now works for me using build
2003-09-17-04 on Windows 2000.

Someone with Linux, could you verify that it works for you and mark the bug
verified?  Thanks.

Comment 9

[Asa Dotzler [:asa]]

verified with today's linux trunk build. 

Comment 10

[Asa Dotzler [:asa]]

not going to hold the release for this. setting approval request to land patch
on 1.5 branch.

Comment 11

[Asa Dotzler [:asa]]

Comment on attachment 131087 [details] [diff] [review]
ignore contents of directory entry reserved field

a=asa (on behalf of drivers) for checkin to the Mozilla 1.5 branch. Please add
the fixed1.5 keyword when this is landed on the branch. Thanks.

Comment 12

[tor]

Checked in on MOZILLA_1_5_BRANCH.

Comment 13

[Christian :Biesinger (don't email me, ping me on IRC)]

Attachment: testcase

icon from the url. seems to be unchanged.