18680 2008-04-22 13:24:48 -0700 Favicon won't load when link tag preceded by script tag 2008-04-23 19:21:21 -0700 1 1 1 Unclassified WebKit New Bugs 528+ (Nightly build) Mac (Intel) OS X 10.5 RESOLVED FIXED InRadar P2 Normal --- 1 simpsora beidson beidson oldest_to_newest 78563 0 simpsora 2008-04-22 13:24:48 -0700 When referencing a favorite icon (favicon) via a link tag, Webkit doesn't display the icon in the URL bar (or even try to load the icon) when the link tag is preceded by a script tag in the source. Instead, it reverts to the default behavior of requesting /favicon.ico. This only happens when the script tag references an external resource with the href attribute. If it's a script block with inline script, the problem doesn't occur. It does not occur when the same script tag appears after the link tag instead of before it. It also does not occur when preceded by a style tag referencing an external resource. I've tested on the following platforms; they all exhibit the problem: OS X 10.4.10 (PPC), Safari 2.0.4 (419.3) OS X 10.5.2 (Intel), Safari 3.1.1 (5525.18) OS X 10.5.2 (Intel), Nightly r32364 My test case uses XHTML 1.0 Transitional, whose DTD specifies that the order of elements in the head section does not matter: (from http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd) <!ENTITY % head.misc "(script|style|meta|link|object|isindex)*"> <!-- content model is %head.misc; combined with a single title and an optional base element in any order --> <!ELEMENT head (%head.misc;, ((title, %head.misc;, (base, %head.misc;)?) | (base, %head.misc;, (title, %head.misc;))))> 78564 1 20754 simpsora 2008-04-22 13:28:18 -0700 Created attachment 20754 Simple testcase exhibiting broken favicon Steps to reproduce: 1. Open favicon_testcase in a affected browser. 2. Note that no favicon appears in URL bar. 3. Edit the testcase, remove the script tag, and save. 4. Refresh the testcase, noting that the favicon now appears in the URL bar. 78569 2 mrowe 2008-04-22 13:47:55 -0700 <rdar://problem/5881376> 78570 3 beidson 2008-04-22 13:50:32 -0700 This is in Radar already, as an example of a real-world site <rdar://problem/5733282> No favicon at washingtonindependent.com 78724 4 beidson 2008-04-23 19:06:20 -0700 This was fixed in r32413 78726 5 simpsora 2008-04-23 19:21:21 -0700 I confirm r32416 fixes both my testcase and my real-world page (and washingtonindependent.com too :)) Thanks. 20754 2008-04-22 13:28:18 -0700 2008-04-22 13:28:18 -0700 Simple testcase exhibiting broken favicon favicon_testcase.html text/html 447 simpsora PCFET0NUWVBFIGh0bWwgUFVCTElDICItLy9XM0MvL0RURCBYSFRNTCAxLjAgVHJhbnNpdGlvbmFs Ly9FTiIgImh0dHA6Ly93d3cudzMub3JnL1RSL3hodG1sMS9EVEQveGh0bWwxLXRyYW5zaXRpb25h bC5kdGQiPgo8aHRtbCB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMTk5OS94aHRtbCIgeG1sOmxh bmc9ImVuIiBsYW5nPSJlbiI+CiAgICA8aGVhZD4KICAgICAgICA8dGl0bGU+U2FmYXJpIGZhdmlj b24gdGVzdDwvdGl0bGU+CiAgICAgICAgPHNjcmlwdCB0eXBlPSd0ZXh0L2phdmFzY3JpcHQnIHNy Yz0nbWlzc2luZy5qcyc+PC9zY3JpcHQ+CiAgICAgICAgPGxpbmsgcmVsPSdzaG9ydGN1dCBpY29u JyBocmVmPSdodHRwOi8vd3d3Lmdvb2dsZS5jb20vZmF2aWNvbi5pY28nIHR5cGU9J2ltYWdlL3gt aWNvbicgLz4KICAgIDwvaGVhZD4KCiAgICA8Ym9keT48L2JvZHk+CjwvaHRtbD4K