From ea176c608f6ce4f4420e129e30e12299e51c8de8 Mon Sep 17 00:00:00 2001 From: Brian Osman Date: Fri, 6 Apr 2018 15:28:23 -0400 Subject: Remove scanlineOrder switch statements that always do the same thing I think these date back to GIF codec having a third value of this enum that meant "interlaced" ? Change-Id: Iaff11a55ad3cff44da2307149991a22447dfe746 Reviewed-on: https://skia-review.googlesource.com/119146 Reviewed-by: Leon Scroggins Commit-Queue: Brian Osman --- dm/DMSrcSink.cpp | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) (limited to 'dm') diff --git a/dm/DMSrcSink.cpp b/dm/DMSrcSink.cpp index 6a14c6363c..4ddebb7e4c 100644 --- a/dm/DMSrcSink.cpp +++ b/dm/DMSrcSink.cpp @@ -616,14 +616,9 @@ Error CodecSrc::draw(SkCanvas* canvas) const { return "Could not start scanline decoder"; } - switch (codec->getScanlineOrder()) { - case SkCodec::kTopDown_SkScanlineOrder: - case SkCodec::kBottomUp_SkScanlineOrder: - // We do not need to check the return value. On an incomplete - // image, memory will be filled with a default value. - codec->getScanlines(dst, height, rowBytes); - break; - } + // We do not need to check the return value. On an incomplete + // image, memory will be filled with a default value. + codec->getScanlines(dst, height, rowBytes); } draw_to_canvas(canvas, bitmapInfo, dst, rowBytes, fDstColorType); -- cgit v1.2.3