[Webkit-unassigned] [Bug 15640] REGRESSION (r17701): Favicons should be viewable as image documents

[bugzilla-daemon at webkit.org]

http://bugs.webkit.org/show_bug.cgi?id=15640


ddkilzer at webkit.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #16930|                            |review?
               Flag|                            |




------- Comment #5 from ddkilzer at webkit.org  2007-10-28 21:11 PDT -------
Created an attachment (id=16930)
 --> (http://bugs.webkit.org/attachment.cgi?id=16930&action=view)
Patch v1

Patch with ChangeLog and layout test.

(In reply to comment #0)
> * NOTES
> The official IANA-registered MIME type for these icons is
> image/vnd.microsoft.icon according to Wikipedia:
> 
> http://en.wikipedia.org/wiki/ICO_(icon_image_file_format)
> 
> The historical MIME type used was image/x-icon.  We should probably support
> both.

Until we have evidence of real web sites returning favicon.ico files with a
MIME type of image/vnd.microsoft.icon, I don't think it's necessary to add this
type.

$ curl --head http://www.microsoft.com/favicon.ico
HTTP/1.1 200 OK
Content-Length: 3638
Content-Type: image/x-icon
Last-Modified: Wed, 11 Oct 2006 22:18:37 GMT
Accept-Ranges: bytes
ETag: "3a71c93283edc61:0"
Server: Microsoft-IIS/7.0
P3P: CP="ALL IND DSP COR ADM CONo CUR CUSo IVAo IVDo PSA PSD TAI TELo OUR SAMo
CNT COM INT NAV ONL PHY PRE PUR UNI"
X-Powered-By: ASP.NET
Date: Mon, 29 Oct 2007 04:10:34 GMT


-- 
Configure bugmail: http://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.