Bug 1711622 Comment 13 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
Chrome 
icon: `image/avif,image/webp,image/apng,image/svg+xml,image/*,*/*;q=0.8`
image: `image/avif,image/webp,image/apng,image/svg+xml,image/*,*/*;q=0.8`

Firefox:
icon: `image/avif,image/jxl,image/webp,*/*`
image: `image/avif,image/jxl,image/webp,*/*

Safari:
icon: `*/*`
image: `image/webp,image/avif,image/jxl,image/heic,image/heic-sequence,video/*;q=0.8,image/png,image/svg+xml,image/*;q=0.8,*/*;q=0.5`

If we were considering adding it only for icon requests, the fix would be to set the Accept header [here](https://searchfox.org/mozilla-central/rev/9bb5d5f55da6cc7163dd93825c25609b769822f9/browser/modules/FaviconLoader.sys.mjs#106).

Otherwise, since Chrome and Firefox both send the same headers for both images and favicon loads, we can probably change the [ImageAcceptHeader](https://searchfox.org/mozilla-central/rev/9bb5d5f55da6cc7163dd93825c25609b769822f9/netwerk/protocol/http/nsHttpHandler.cpp#208)

@Simon, do you have any thoughts? Should we just add `image/svg+xml` to the image accept header unconditionally?
Revision 1 by
on 
Chrome 
icon: `image/avif,image/webp,image/apng,image/svg+xml,image/*,*/*;q=0.8`
image: `image/avif,image/webp,image/apng,image/svg+xml,image/*,*/*;q=0.8`

Firefox:
icon: `image/avif,image/jxl,image/webp,*/*`
image: `image/avif,image/jxl,image/webp,*/*`

Safari:
icon: `*/*`
image: `image/webp,image/avif,image/jxl,image/heic,image/heic-sequence,video/*;q=0.8,image/png,image/svg+xml,image/*;q=0.8,*/*;q=0.5`

If we were considering adding it only for icon requests, the fix would be to set the Accept header [here](https://searchfox.org/mozilla-central/rev/9bb5d5f55da6cc7163dd93825c25609b769822f9/browser/modules/FaviconLoader.sys.mjs#106).

Otherwise, since Chrome and Firefox both send the same headers for both images and favicon loads, we can probably change the [ImageAcceptHeader](https://searchfox.org/mozilla-central/rev/9bb5d5f55da6cc7163dd93825c25609b769822f9/netwerk/protocol/http/nsHttpHandler.cpp#208)

@Simon, do you have any thoughts? Should we just add `image/svg+xml` to the image accept header unconditionally?

Back to Bug 1711622 Comment 13