Index: third_party/WebKit/LayoutTests/canvas/philip/tests/2d.drawImage.animated.poster.html |
diff --git a/third_party/WebKit/LayoutTests/canvas/philip/tests/2d.drawImage.animated.poster.html b/third_party/WebKit/LayoutTests/canvas/philip/tests/2d.drawImage.animated.poster.html |
index 123b890883f699c17a1ace889fe81daee0810fdb..7bbae88e78375e365e55aaebdf5d3e5a71d49b12 100644 |
--- a/third_party/WebKit/LayoutTests/canvas/philip/tests/2d.drawImage.animated.poster.html |
+++ b/third_party/WebKit/LayoutTests/canvas/philip/tests/2d.drawImage.animated.poster.html |
@@ -12,8 +12,24 @@ |
<script> |
_addTest(function(canvas, ctx) { |
-ctx.drawImage(document.getElementById('anim-poster-gr.png'), 0, 0); |
-_assertPixelApprox(canvas, 50,25, 0,255,0,255, "50,25", "0,255,0,255", 2); |
+// This test is disabled because it is wrong. It expects to draw the IDAT frame |
+// of an animated PNG in which the IDAT frame is not part of the animation. This |
+// should only happen if we do *not* support APNG. Now that we support APNG, |
+// this test fails. |
+// |
+// The ImageDecoder is not aware of what method is requesting the decode, and |
+// the APNG specification [1] does explicitly state that the IDAT is not part |
+// of the animated image if there's no fcTL chunk preceding it. It would make |
+// more sense if this test would verify that the first frame of the image is |
+// drawn, but that is explicitly not the purpose, as stated at [2]. In Firefox, |
+// the developer of the APNG specification, this test [2] fails too. |
+// |
+// [1] https://wiki.mozilla.org/APNG_Specification |
+// [2] http://www.w3c-test.org/2dcontext/drawing-images-to-the-canvas/2d.drawImage.animated.apng.html |
+ _assertSame("disabled", "disabled", "disabled", "disabled"); |
+ |
+// ctx.drawImage(document.getElementById('anim-poster-gr.png'), 0, 0); |
+// _assertPixelApprox(canvas, 50,25, 0,255,0,255, "50,25", "0,255,0,255", 2); |
}); |