Bug 120784 - REGRESSION (r122806): A different favicon is chosen from multiple ones
Summary: REGRESSION (r122806): A different favicon is chosen from multiple ones
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.8
: P2 Normal
Assignee: Nobody
URL: http://dev.visuallizard.com/demo/favi...
Keywords: Regression
Depends on:
Blocks:
 
Reported: 2013-09-05 11:58 PDT by Maxim
Modified: 2013-09-09 10:09 PDT (History)
4 users (show)

See Also:


Attachments
Screenshots of the issue in different browsers (263.92 KB, image/png)
2013-09-09 09:15 PDT, Maxim
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Maxim 2013-09-05 11:58:25 PDT
The 32x32 pixels favicons are not displayed properly anymore. The icon is squished and pushed to the right and it looks like there is something added to it while page loads. The issue occurs on latest WebKit r154944 and on Safari 6.1 seed 7. However older Safari 6.0 does not have this issue. The "sizes" attribute does not seem to make any effect on it either.
Comment 1 Alexey Proskuryakov 2013-09-05 12:54:36 PDT
Thank you for the report.

Do you have a test URL or a test case that you could attach?
Comment 2 Maxim 2013-09-09 09:15:21 PDT
Created attachment 211049 [details]
Screenshots of the issue in different browsers

Now when I had some free time for more thorough tests it looks like the issues is not related to 32x32 pixel icon, but rather to the fact how it is being loaded by:

<link rel="icon" href="logo.png" type="image/png" />
<link rel="shortcut icon" href="favicon.ico?v=4" type="image/x-icon" />

Webkit uses only first tag and loads logo.png, although other browsers load favicon.ico . Perhaps my markup is wrong, but I cant seem to recall any problems. Feel free to check demo at http://dev.visuallizard.com/demo/favicon/ and screenshot attached. I should note that IE11 has similar results as Webkit.
Could someone with better understanding of this compare test case with the specs please?

PS. Should I close the bug report, since it's title isn't relevant anymore?

Thanks, and sorry for confusion,
Max
Comment 3 Alexey Proskuryakov 2013-09-09 10:02:50 PDT
This changed with <http://trac.webkit.org/changeset/122806> (bug 88665). Reading that bug, this is clearly an unintended consequence of that change, but I don't know if the new behavior is correct or not.

Could you please keep the demo at <http://dev.visuallizard.com/demo/favicon/> online while we are investigating?
Comment 4 Maxim 2013-09-09 10:09:03 PDT
Sure, no problem. let me know if you need more tests or changes from my end.
Thanks Alexey