Scanline decoding for bmp

src/codec/SkBmpMaskCodec.cpp

 /*
  * Copyright 2015 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkBmpMaskCodec.h"
 #include "SkCodecPriv.h"
 #include "SkColorPriv.h"
@@ skipping 22 matching lines @@
         default:
             return false;
     }
 }
 
 
 /*
  * Creates an instance of the decoder
  */
 SkBmpMaskCodec::SkBmpMaskCodec(const SkImageInfo& info, SkStream* stream,
+                               SkBmpCodec::BmpInputFormat inputFormat,
                                uint16_t bitsPerPixel, SkMasks* masks,
                                SkBmpCodec::RowOrder rowOrder)
-    : INHERITED(info, stream, bitsPerPixel, rowOrder)
+    : INHERITED(info, stream, inputFormat, bitsPerPixel, rowOrder)
     , fMasks(masks)
     , fMaskSwizzler(NULL)
     , fSrcBuffer(NULL)
 {}
 
 /*
  * Initiates the bitmap decode
  */
 SkCodec::Result SkBmpMaskCodec::onGetPixels(const SkImageInfo& dstInfo,
                                             void* dst, size_t dstRowBytes,
@@ skipping 72 matching lines @@
 
         // Decode the row in destination format
         int row = SkBmpCodec::kBottomUp_RowOrder == this->rowOrder() ? height - 1 - y : y;
         void* dstRow = SkTAddOffset<void>(dst, row * dstRowBytes);
         fMaskSwizzler->swizzle(dstRow, srcRow);
     }
 
     // Finished decoding the entire image
     return kSuccess;
 }
+
+SkBmpMaskScanlineDecoder::SkBmpMaskScanlineDecoder(SkBmpMaskCodec* codec)
+    : INHERITED(codec->getInfo())
+    , fCodec(codec)
+{}
+
+SkCodec::Result SkBmpMaskScanlineDecoder::onStart(const SkImageInfo& dstInfo,
+        const SkCodec::Options& options, SkPMColor inputColorPtr[], int* inputColorCount) {
+    if (!fCodec->rewindIfNeeded()) {
+        return SkCodec::kCouldNotRewind;
+    }
+    if (options.fSubset) {
+        // Subsets are not supported.
+        return SkCodec::kUnimplemented;
+    }
+    if (dstInfo.dimensions() != fCodec->getInfo().dimensions()) {
+        SkCodecPrintf("Error: scaling not supported.\n");
+        return SkCodec::kInvalidScale;
+    }
+
+    if (!conversion_possible(dstInfo, fCodec->getInfo())) {
+        SkCodecPrintf("Error: cannot convert input type to output type.\n");
+        return SkCodec::kInvalidConversion;
+    }
+
+    // Initialize a the mask swizzler
+    if (!fCodec->initializeSwizzler(dstInfo)) {
+        SkCodecPrintf("Error: cannot initialize swizzler.\n");
+        return SkCodec::kInvalidConversion;
+    }
+
+    fDstInfo = dstInfo;

[scroggo, 2015/08/13 21:25:36]

Can you get this from the base class, which also stores it?

Aha - it looks like you are, and you never use this copy of it. Please remove.

[msarett, 2015/08/14 15:44:48]

Done.

+    fOpts = options;

[scroggo, 2015/08/13 21:25:36]

Maybe this should be stored in the base class as well?

[msarett, 2015/08/14 15:44:48]

fOptions is now stored in the base class.

+
+    return SkCodec::kSuccess;
+}
+
+SkCodec::Result SkBmpMaskScanlineDecoder::onGetScanlines(void* dst,
+        int count, size_t rowBytes) {
+    // Create a new image info representing the portion of the image to decode
+    SkImageInfo rowInfo = fDstInfo.makeWH(dstInfo().width(), count);

[scroggo, 2015/08/13 21:25:35]

nit: this->dstInfo()...

[msarett, 2015/08/14 15:44:48]

Done.

+
+    // Decode the requested rows
+    SkCodec::Result r = fCodec->decode(rowInfo, dst, rowBytes, fOpts);
+}