Fix encoding

Author: JimBobSquarePants

src/ImageSharp/Formats/Png/PngEncoderCore.cs

@@ -189,6 +189,7 @@ public void Encode<TPixel>(Image<TPixel> image, Stream stream, CancellationToken
             ReadOnlyMemory<TPixel>? previousPalette = quantized?.Palette.ToArray();
 
             // Write following frames.
+            uint increment = 0;
             for (int i = 1; i < image.Frames.Count; i++)
             {
                 currentFrame = image.Frames[i];
@@ -200,12 +201,14 @@ public void Encode<TPixel>(Image<TPixel> image, Stream stream, CancellationToken
                     ClearTransparentPixels(currentFrame);
                 }
 
-                frameControl = this.WriteFrameControlChunk(stream, currentFrame, (uint)i);
+                // Each frame control sequence number must be incremented by the
+                // number of frame data chunks that follow.
+                frameControl = this.WriteFrameControlChunk(stream, currentFrame, (uint)i + increment);
 
                 // Dispose of previous quantized frame and reassign.
                 quantized?.Dispose();
                 quantized = this.CreateQuantizedImageAndUpdateBitDepth(pngMetadata, currentFrame, previousPalette);
-                this.WriteDataChunks(frameControl, currentFrame, quantized, stream, true);
+                increment += this.WriteDataChunks(frameControl, currentFrame, quantized, stream, true);
             }
         }
         else
@@ -1013,7 +1016,7 @@ private FrameControl WriteFrameControlChunk(Stream stream, ImageFrame imageFrame
     /// <param name="quantized">The quantized pixel data. Can be null.</param>
     /// <param name="stream">The stream.</param>
     /// <param name="isFrame">Is writing fdAT or IDAT.</param>
-    private int WriteDataChunks<TPixel>(FrameControl frameControl, ImageFrame<TPixel> pixels, IndexedImageFrame<TPixel>? quantized, Stream stream, bool isFrame)
+    private uint WriteDataChunks<TPixel>(FrameControl frameControl, ImageFrame<TPixel> pixels, IndexedImageFrame<TPixel>? quantized, Stream stream, bool isFrame)
         where TPixel : unmanaged, IPixel<TPixel>
     {
         byte[] buffer;
@@ -1070,7 +1073,9 @@ private int WriteDataChunks<TPixel>(FrameControl frameControl, ImageFrame<TPixel
 
             if (isFrame)
             {
-                uint sequenceNumber = (uint)(frameControl.SequenceNumber + i);
+                // We increment the sequence number for each frame chunk.
+                // '1' is added to the sequence number to account for the preceding frame control chunk.
+                uint sequenceNumber = (uint)(frameControl.SequenceNumber + 1 + i);
                 this.WriteFrameDataChunk(stream, sequenceNumber, buffer, i * maxBlockSize, length);
             }
             else
@@ -1079,7 +1084,7 @@ private int WriteDataChunks<TPixel>(FrameControl frameControl, ImageFrame<TPixel
             }
         }
 
-        return numChunks;
+        return (uint)numChunks;
     }
 
     /// <summary>

tests/ImageSharp.Tests/Formats/Png/PngDecoderTests.cs

@@ -111,7 +111,12 @@ public void Decode<TPixel>(TestImageProvider<TPixel> provider)
     public void Decode_APng<TPixel>(TestImageProvider<TPixel> provider)
         where TPixel : unmanaged, IPixel<TPixel>
     {
-        using Image<TPixel> image = provider.GetImage(PngDecoder.Instance); // MagickReferenceDecoder cannot decode APNGs
+        using Image<TPixel> image = provider.GetImage(PngDecoder.Instance);
+
+        Assert.Equal(5, image.Frames.Count);
+
+        // TODO: Assertations.
+        // MagickReferenceDecoder cannot decode APNGs (Though ImageMagick can, we likely need to update our mapping implementation)
     }
 
     [Theory]

tests/ImageSharp.Tests/Formats/Png/PngEncoderTests.cs

@@ -451,8 +451,14 @@ public void Encode_APng<TPixel>(TestImageProvider<TPixel> provider)
         using MemoryStream memStream = new();
         image.Save(memStream, PngEncoder);
         memStream.Position = 0;
+
+        image.DebugSave(provider: provider, encoder: PngEncoder, null, false);
+
         using Image<Rgba32> output = Image.Load<Rgba32>(memStream);
         ImageComparer.Exact.VerifySimilarity(output, image);
+
+        // TODO: Additional assertations regarding metadata.
+        Assert.Equal(5, image.Frames.Count);
     }
 
     [Theory]