Ensure delays of the origin image is preserved (#3062)

By not initializing the delay vector, since we assume it's empty by default. See: #3061.
lovell · Jan 28, 2022 · 7aa3402 · 7aa3402
1 parent 68823a5
commit 7aa3402
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/src/pipeline.h b/src/pipeline.h
@@ -327,7 +327,6 @@ struct PipelineBaton {
     ensureAlpha(-1.0),
     colourspaceInput(VIPS_INTERPRETATION_LAST),
     colourspace(VIPS_INTERPRETATION_LAST),
-    delay{-1},
     loop(-1),
     tileSize(256),
     tileOverlap(0),

diff --git a/test/unit/webp.js b/test/unit/webp.js
@@ -183,6 +183,15 @@ describe('WebP', function () {
     assert.deepStrictEqual(updated.delay, expectedDelay);
   });
 
+  it('should preserve delay between frames', async () => {
+    const updated = await sharp(fixtures.inputWebPAnimated, { pages: -1 })
+      .webp()
+      .toBuffer()
+      .then(data => sharp(data, { pages: -1 }).metadata());
+
+    assert.deepStrictEqual(updated.delay, [120, 120, 90, 120, 120, 90, 120, 90, 30]);
+  });
+
   it('should work with streams when only animated is set', function (done) {
     fs.createReadStream(fixtures.inputWebPAnimated)
       .pipe(sharp({ animated: true }))