tree 4a0d271bbb2eb9561be97b87187a79bc075e407a
parent c890df13026caaa7a52298c9357f85f608e78f32
author Lukasz Anforowicz <lukasza@google.com> 1761355926 -0700
committer Pdfium LUCI CQ <pdfium-scoped@luci-project-accounts.iam.gserviceaccount.com> 1761355926 -0700

[rust png] Add explicit `PngFinishedDecoding` callback.

Before this CL, `ProgressiveDecoder` would keep calling into
`PngDecoder::ContinueDecode` until consuming the whole `file_`.  After
this CL, `ProgressiveDecoder` relies on the new `PngFinishedDecoding`
callback instead.

This approach is:

* Consistent with how other codecs work.  For example see how the loops
  in `ProgressiveDecoder::BmpContinueDecode` and/or
  `ProgressiveDecoder::GifContinueDecode` continue until the codec
  indicates that the full image has been decoded.
* More robust - for example a truncated input should not anymore be
  treated as a success (before this CL the new `TruncatedPng` unit tests
  would have ended with `kDecodeFinished` rather than `kError`).

Bug: 444045690
Change-Id: Iffbe0d2e36471af9ab5a0157fd19abefa7f01c75
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/137274
Reviewed-by: Lei Zhang <thestig@chromium.org>
Auto-Submit: Łukasz Anforowicz <lukasza@google.com>
Commit-Queue: Lei Zhang <thestig@chromium.org>