/[blink]
Chromium logo

Revision 188423


Jump to revision: Previous Next
Author: hclam@chromium.org
Date: Wed Jan 14 23:28:11 2015 UTC (8 years, 4 months ago)
Changed paths: 4
Log Message:
Fix handling of broken GIFs with weird frame sizes

Code didn't handle well if a GIF frame has dimension greater than the
"screen" dimension. This will break deferred image decoding.

This change reports the size as final only when the first frame is
encountered.

Added a test to verify this behavior. Frame size reported by the decoder
should be constant.

BUG=437651
R=pkasting@chromium.org, senorblanco@chromium.org

Review URL: https://codereview.chromium.org/813943003

Changed paths

Path Details
Directorytrunk/Source/platform/image-decoders/gif/GIFImageDecoderTest.cpp modified , text changed
Directorytrunk/Source/platform/image-decoders/gif/GIFImageReader.cpp modified , text changed
Directorytrunk/Source/platform/image-decoders/gif/GIFImageReader.h modified , text changed
Directorytrunk/Source/platform/image-decoders/testing/first-frame-has-greater-size-than-screen-size.gif added

Powered by ViewVC 1.1.26 ViewVC Help