[Cocoa] Support AVIF images for macOS Ventura and iOS 16 - #1717
Conversation
There was a problem hiding this comment.
uh oh. Why is there both HAVE(AVIF) and USE(AVIF)? Can we consolidate them?
There was a problem hiding this comment.
HAVE(AVIF) is used by Apple ports where the system frameworks provide the needed support to render these images.
USE(AVIF) is used by non Apple ports. It is defined in Source/cmake/OptionsWPE.cmake and Source/cmake/OptionsGTK.cmake. They use libavif to handle the AVIF images. If we want to merge HAVE(AVIF) and USE(AVIF) and merge HAVE(WEBP) and USE(WEBP), I think it is better to handle this in a separate patch.
There was a problem hiding this comment.
Can we make sure we have a bug open for the separate patch? (Presumably the separate patch would be for both AVIF and WEBP.)
There was a problem hiding this comment.
120c22a to
67b4e14
Compare
There was a problem hiding this comment.
Still not super clear if this line has any repercussions beyond HEIC or AVIF, but okay.
https://bugs.webkit.org/show_bug.cgi?id=241904 rdar://95742091 Reviewed by Myles C. Maxfield. Add the mime type and the UTI of the AVIF to the list of the allowed image formats. The system frameworks will be used to render the AVIF images on macOS Ventura and iOS 16. Because of sand-boxing limitations, software decoding has to be used for AVIF images. * LayoutTests/platform/mac/TestExpectations: * Source/WTF/wtf/PlatformHave.h: * Source/WebCore/loader/cache/CachedResourceRequest.cpp: (WebCore::acceptHeaderValueForImageResource): * Source/WebCore/platform/MIMETypeRegistry.cpp: * Source/WebCore/platform/graphics/cg/ImageDecoderCG.cpp: (WebCore::createImageSourceOptions): * Source/WebCore/platform/graphics/cg/UTIRegistry.cpp: (WebCore::defaultSupportedImageTypes): Canonical link: https://commits.webkit.org/251850@main
67b4e14 to
59d774a
Compare
|
Committed 251850@main (59d774a): https://commits.webkit.org/251850@main Reviewed commits have been landed. Closing PR #1717 and removing active labels. |
59d774a