WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED DUPLICATE of
bug 12147
12002
<link rel="icon"> is not loaded or displayed on individual HTML pages
https://bugs.webkit.org/show_bug.cgi?id=12002
Summary
<link rel="icon"> is not loaded or displayed on individual HTML pages
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.
Top of Page
Format For Printing
XML
Clone This Bug