Bug 120784

Summary: REGRESSION (r122806): A different favicon is chosen from multiple ones
Product: WebKit Reporter: Maxim <maxim.chiriac>
Component: WebKit Misc.Assignee: Nobody <webkit-unassigned>
Status: NEW    
Severity: Normal CC: ap, beidson, petewil, tkent
Priority: P2 Keywords: Regression
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.8   
URL: http://dev.visuallizard.com/demo/favicon/
Attachments:
Description Flags
Screenshots of the issue in different browsers none

Maxim
Reported 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.
Attachments
Screenshots of the issue in different browsers (263.92 KB, image/png)
2013-09-09 09:15 PDT, Maxim 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Alexey Proskuryakov
Comment 1 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?
Maxim
Comment 2 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
Alexey Proskuryakov
Comment 3 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?
Maxim
Comment 4 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
Note You need to log in before you can comment on or make changes to this bug.