[Skia] Animation on images motion mark test doesn't work on the CPU

https://bugs.webkit.org/show_bug.cgi?id=271319 Reviewed by Nikolas Zimmermann. Use SkSurface::peekPixels() instead of SkSurface::makeImageSnapshot() when creating a NativeImage reference. * Source/WebCore/platform/graphics/skia/ImageBufferSkiaUnacceleratedBackend.cpp: (WebCore::ImageBufferSkiaUnacceleratedBackend::createNativeImageReference): Canonical link: https://commits.webkit.org/276448@main
WebKit · Mar 21, 2024 · f349ceb · f349ceb
1 parent 82fe811
commit f349ceb
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/Source/WebCore/platform/graphics/skia/ImageBufferSkiaUnacceleratedBackend.cpp b/Source/WebCore/platform/graphics/skia/ImageBufferSkiaUnacceleratedBackend.cpp
@@ -69,7 +69,13 @@ RefPtr<NativeImage> ImageBufferSkiaUnacceleratedBackend::copyNativeImage()
 
 RefPtr<NativeImage> ImageBufferSkiaUnacceleratedBackend::createNativeImageReference()
 {
-    return NativeImage::create(m_surface->makeImageSnapshot());
+    SkPixmap pixmap;
+    if (m_surface->peekPixels(&pixmap)) {
+        return NativeImage::create(SkImages::RasterFromPixmap(pixmap, [](const void*, void* context) {
+            static_cast<SkSurface*>(context)->unref();
+        }, SkSafeRef(m_surface.get())));
+    }
+    return nullptr;
 }
 
 void ImageBufferSkiaUnacceleratedBackend::getPixelBuffer(const IntRect& srcRect, PixelBuffer& destination)