Fill incomplete images in SkCodec parent class

src/codec/SkMaskSwizzler.h

Index: src/codec/SkMaskSwizzler.h
diff --git a/src/codec/SkMaskSwizzler.h b/src/codec/SkMaskSwizzler.h
index fbc951a070806e954896e56913e863360ca398ee..20b6e2dbe2d476b495acaea6342661f517fd0426 100644
--- a/src/codec/SkMaskSwizzler.h
+++ b/src/codec/SkMaskSwizzler.h
@@ -35,6 +35,15 @@ public:
      */
     SkSwizzler::ResultAlpha swizzle(void* dst, const uint8_t* SK_RESTRICT src);
 
+    /**
+     * Implement fill using a custom width.
+     */
+    void fill(void* dst, const SkImageInfo& info, size_t rowBytes, uint32_t colorOrIndex,

[scroggo, 2015/10/08 13:50:30]

Again, I think this parameter order should match getPixels (info, dst, rowbytes)

Also, it seems weird that this function needs to be passed so much information.
For example, it was created with a particular color type/alpha type, so it
should only use that one (although I suppose it does not store that info). I
suppose this allows it to call SkSampler::Fill and share code.

[msarett, 2015/10/08 15:33:25]

Changed the parameter order.

Yeah on the opposite extreme, the Swizzler could store colorType, alphaType,
zeroInit, and (maybe) rowBytes.  Then we would only need to pass in dst, numRow,
and colorOrIndex.

+            SkCodec::ZeroInitialized zeroInit) override {
+        const SkImageInfo fillInfo = info.makeWH(fDstWidth, info.height());
+        SkSampler::Fill(dst, fillInfo, rowBytes, colorOrIndex, zeroInit);
+    }
+
 private:
 
     /*