commit | 8c6df8cd99af205f85a8884f6594d38d21f51b82 | [log] [tgz] |
---|---|---|
author | Nigel Tao <nigeltao@google.com> | Tue Nov 23 22:51:09 2021 +1100 |
committer | SkCQ <skcq-be@skia-corp.google.com.iam.gserviceaccount.com> | Wed Nov 24 16:45:43 2021 +0000 |
tree | 53addd01bfccdbc63892e06e5fc68e4e67342b44 | |
parent | b1f450bb2bb86af3fc3daaf7feed8fe0962c789a [diff] |
Fix convert-to-NIA stillImage check For example, this GIF image: https://github.com/google/wuffs/blob/e44920d/test/data/artificial-gif/pixel-data-too-much-bad-lzw.gif which is 'disassembled' here: https://github.com/google/wuffs/blob/e44920d/test/data/artificial-gif/pixel-data-too-much-bad-lzw.gif.make-artificial.txt only has one frame, regardless of how the GIF codec implementation deals with the LZW-compression error. It is not an animated image. Nonetheless, frameInfos.size() returns 1, not 0, so prior to this commit, "stillImage = frameInfos.empty()" assigned false, not true. See also this Wuffs (not Skia) commit: https://github.com/google/wuffs/commit/3a9933f2a0af846affe15eb6f10ce26b029bd7a8 Bug: None Change-Id: Ia69a486abe5c2caa7488c55a37c403e7660ff168 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/475436 Commit-Queue: Leon Scroggins <scroggo@google.com> Reviewed-by: Leon Scroggins <scroggo@google.com>