commit 79ad95c658586ec242e6f97bcd084ea7676a86c7
author Lukasz Anforowicz <lukasza@chromium.org> Fri Aug 30 23:39:45 2024 +0000
committer SkCQ <skcq-be@skia-corp.google.com.iam.gserviceaccount.com> Tue Sep 03 15:13:26 2024 +0000
tree 1389de95ba3d99aeb78a210c1ae61839189ea1f4
parent 2d5a75027691e0fe93c84407b159d08359cc0b0d

[rust png] Minimal `onGetFrameInfo` to support non-animated images.

This CL adds a miminal `SkPngRustCodec::onGetFrameInfo` - one that only
supports the first frame (i.e. only supports non-animated images).  This
is required (although not sufficient) to unblock Blink integration of
`SkPngRustCodec` - this integration will reuse code from
`blink::GIFImageDecoder` which depends on `onGetFrameInfo` calls).

After this CL, the following Blink tests pass (after patching in the
Blink integration from https://crrev.com/c/5786777 and upstream CLs):

* `RustEnabled/PNGTests.sizeMayOverflow/0`
* `RustEnabled/PNGTests.truncated/0`
* `RustEnabled/PNGTests.crbug827754/0`
* `RustEnabled/PNGTests.CriticalPrivateChunkBeforeIHDR/0`

Bug: chromium:356884491
Change-Id: I9cc80a657aea624d37c327ba7bffbad2694e6f61
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/888898
Commit-Queue: Łukasz Anforowicz <lukasza@google.com>
Reviewed-by: Brian Osman <brianosman@google.com>