Skip to content

Conversation

@JimBobSquarePants
Copy link
Member

Prerequisites

  • I have written a descriptive pull-request title
  • I have verified that there are no overlapping pull-requests open
  • I have verified that I am following the existing coding patterns and practice as demonstrated in the repository. These follow strict Stylecop rules 👮.
  • I have provided test coverage for my change (where applicable)

Description

Fixes #2992
Fixes #2988 (I wasn't using up my LFS budget for a separate run)

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds comprehensive metadata parsing support for WebP animated images, addressing issues #2992 and #2988. The changes enable full frame metadata extraction during the Identify operation for animated WebP files, bringing animation metadata handling to parity with full decode operations.

Key Changes:

  • Implemented Identify method in WebpAnimationDecoder to extract frame metadata without full image decoding
  • Enhanced segment integrity handling with configurable validation for EXIF/XMP chunks
  • Added resolution metadata extraction from EXIF profiles during WebP decoding

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
WebpDecoderTests.cs Added test cases to verify frame metadata extraction via Identify for both lossless and lossy animated WebP images
WebpImageInfo.cs Added DataSize property to track encoded image data size
WebpDecoderCore.cs Integrated animation decoder into Identify path and added configurable segment integrity handling
WebpChunkParsingUtils.cs Enhanced optional chunk parsing with EXIF resolution extraction and segment integrity validation
WebpAnimationDecoder.cs Implemented new Identify method and added ReadFrameInfo helper for metadata-only parsing
PngFrameMetadata.cs Fixed documentation error: changed "hundredths" to "seconds" for frame delay description

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@JimBobSquarePants JimBobSquarePants merged commit 44f5af1 into main Oct 23, 2025
9 checks passed
@JimBobSquarePants JimBobSquarePants deleted the js/fix-2992 branch October 23, 2025 23:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ImageInfo.FrameMetadataCollection not populated for animated webp images Incorrect XMLDoc of APNG

2 participants