Bug 12002

Summary: <link rel="icon"> is not loaded or displayed on individual HTML pages
Product: WebKit Reporter: David Kilzer (:ddkilzer) <ddkilzer>
Component: Page LoadingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED DUPLICATE    
Severity: Enhancement CC: beidson
Priority: P2 Keywords: HasReduction
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
Test page none

David Kilzer (:ddkilzer)
Reported 2006-12-27 10:31:33 PST
Summary: When loading an HTML page with a <link rel="icon"> tag in the <head>, Safari doesn't load this icon and doesn't display it in the address bar. Both Opera 9 and Firefox 2 display this icon properly. Not tested on MSIE 6 yet. Steps to reproduce: 1. Open Safari. 2. Load test page. Expected results: The webkit.org favicon should be loaded (icon should appear in the Activity Window) and displayed in the address bar when the page is loaded. Actual results: The favicon is not loaded and is not displayed in the address bar when the page is loaded. Regression: Shipping Safari 2.0.4 (419.3) on Mac OS X 10.4.8 (8N1037) exhibits the actual results, so this is not a regression. Tested with a locally-built debug build of WebKit r18423. Notes: Code in WebKit/DOM/WebDOMOperations.m for DOMHTMLLinkElement leads me to believe that this icon is expected to be loaded (or was loaded at one time) since saving a WebArchive uses this code path and will try to save the icon to the WebArchive file (and fail).
Attachments
Test page (1.38 KB, text/html)
2006-12-27 10:32 PST, David Kilzer (:ddkilzer) 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
David Kilzer (:ddkilzer)
Comment 1 2006-12-27 10:32:30 PST
Created attachment 12066 [details] Test page
David Kilzer (:ddkilzer)
Comment 2 2006-12-28 16:08:08 PST
Once Bug 11882 lands, this fix will need a webarchive test: <link rel="icon" href="resources/favicon.ico" type="image/x-icon">
David Kilzer (:ddkilzer)
Comment 3 2007-01-08 10:46:19 PST
*** This bug has been marked as a duplicate of 12147 ***
Note You need to log in before you can comment on or make changes to this bug.