Swizzler changes Index8 and 565

src/codec/SkSwizzler.h

Index: src/codec/SkSwizzler.h
diff --git a/src/codec/SkSwizzler.h b/src/codec/SkSwizzler.h
index 63afbf73bc3d75717f20dc8650f64e3f54d29425..d54c58b0c735b935c79e1fcbf8d3b9d2162a2c12 100644
--- a/src/codec/SkSwizzler.h
+++ b/src/codec/SkSwizzler.h
@@ -128,6 +128,17 @@ public:
                                       const SkImageInfo&, void* dst,
                                       size_t dstRowBytes,
                                       SkImageGenerator::ZeroInitialized);
+
+    /**
+     * Fill the remainder of the destination with a single color
+     * @param y            the starting row for the fill
+     * @param input        the value to fill with - may be a color or an index

[scroggo, 2015/04/06 14:55:12]

Maybe expand this comment to explain why it might be a color vs an index, and
when it will be treated as a uint8_t.

[msarett, 2015/04/06 18:10:56]

Done.

+     * @param colorTable   a non-NULL colorTable indicates that the input is an index
+     *                     even if the dst color type is not an index type
+     */
+    static void Fill(void* dst, const SkImageInfo& dstInfo, size_t dstRowBytes, uint32_t y,
+            uint32_t input, SkPMColor* colorTable);

[scroggo, 2015/04/06 14:55:12]

I find it a little surprising that we do not need to know the SrcConfig. I guess
we always convert our colors to 32bit before here (unless it's an index)?

[msarett, 2015/04/06 18:10:56]

Yeah you are right.  It works because we are always expecting kN32 colors for
the destination (unless its an index, indicated by the color table), so we know
that is what will be passed in.

We may need to add srcConfig is we end up adding 565 or something like that
here.

+
     /**
      *  Swizzle the next line. Call height times, once for each row of source.
      *  @param src The next row of the source data.
@@ -156,6 +167,7 @@ public:
      *  destination?
      */
     void setDstRow(void* dst) { fDstRow = dst; }
+
 private:
 
 #ifdef SK_DEBUG