183257 2018-03-01 13:24:49 -0800 [WPE] Add support for WebP encoding in HTMLCanvasElement 2019-08-25 18:57:18 -0700 1 1 1 Unclassified WebKit WPE WebKit WebKit Nightly Build Unspecified Unspecified NEW P2 Normal --- 1 cadubentzen webkit-unassigned bugs-noreply franklinyu me oldest_to_newest 1403012 0 cadubentzen 2018-03-01 13:24:49 -0800 Currently LayoutTests/fast/canvas/canvas-toDataURL-webp.html fails. I think the path to implement it is: - Add WebPImageEncoder.{h,cpp} to Source/WebCore/platform/image-encoders and implement compressRGBABigEndianToWebP() using libwebp - Extend encodeImage() in Source/WebCore/platform/graphics/cairo/ImageBufferCairo.cpp to use the encoder implemented. - Add WebP encoding to the MIMETypeRegistry - Find encode.h as well in FindWebP.cmake Is that the right path? I would happily implement this if so. Also, is this feature relevant? I couldn't find the requirement on W3C and I think right now only Chrome implements it. 1413925 1 franklinyu 2018-04-11 21:44:47 -0700 I believe this is duplicate of https://bugs.webkit.org/show_bug.cgi?id=125499. 1413926 2 cadubentzen 2018-04-11 21:57:56 -0700 Yes. It is the same test failing. However, I mentioned here specifically the WPE port. It did it because the image encoding code is different from other ports. For example, for GTK it is implemented in Source/WebCore/platform/graphics/gtk/ImageBufferGtk.cpp, relying on GdkPixBuf. As this code isn't shared between ports I opened this one for a possible implementation only for WPE. For GTK it could be added a fallback to reuse the implementation intended here using libwebp in the special case of WebP images. 1413927 3 cadubentzen 2018-04-11 21:59:01 -0700 > It did it typo: I did it