Implement Fill() for incomplete decodes to RGBA_F16

include/codec/SkCodec.h

 /*
  * Copyright 2015 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef SkCodec_DEFINED
 #define SkCodec_DEFINED
 
@@ skipping 583 matching lines @@
      *  Subclasses should do any set up needed after a rewind.
      */
     virtual bool onRewind() {
         return true;
     }
 
     /**
      * On an incomplete input, getPixels() and getScanlines() will fill any uninitialized
      * scanlines.  This allows the subclass to indicate what value to fill with.
      *
-     * @param colorType Destination color type.
+     * @param dstInfo   Describes the destination.
      * @return          The value with which to fill uninitialized pixels.
      *
-     * Note that we can interpret the return value as an SkPMColor, a 16-bit 565 color,
-     * an 8-bit gray color, or an 8-bit index into a color table, depending on the color
-     * type.
+     * Note that we can interpret the return value as a 64-bit Float16 color, a SkPMColor,
+     * a 16-bit 565 color, an 8-bit gray color, or an 8-bit index into a color table,
+     * depending on the color type.
      */
-    uint32_t getFillValue(SkColorType colorType) const {
-        return this->onGetFillValue(colorType);
+    uint64_t getFillValue(const SkImageInfo& dstInfo) const {
+        return this->onGetFillValue(dstInfo);
     }
 
     /**
      * Some subclasses will override this function, but this is a useful default for the color
      * types that we support.  Note that for color types that do not use the full 32-bits,
      * we will simply take the low bits of the fill value.
      *
+     * kRGBA_F16_SkColorType: Transparent or Black, depending on the src alpha type

[reed1, 2016/09/13 14:41:29]

Is this list your "default" values? Not clear from the comment.

[msarett, 2016/09/13 14:47:24]

Improving comments.

      * kN32_SkColorType: Transparent or Black, depending on the src alpha type
      * kRGB_565_SkColorType: Black
      * kGray_8_SkColorType: Black
      * kIndex_8_SkColorType: First color in color table
      */
-    virtual uint32_t onGetFillValue(SkColorType /*colorType*/) const {
-        return kOpaque_SkAlphaType == fSrcInfo.alphaType() ? SK_ColorBLACK : SK_ColorTRANSPARENT;
-    }
+    virtual uint64_t onGetFillValue(const SkImageInfo& dstInfo) const;
 
     /**
      * Get method for the input stream
      */
     SkStream* stream() {
         return fStream.get();
     }
 
     /**
      *  The remaining functions revolve around decoding scanlines.
@@ skipping 92 matching lines @@
 
     // For testing with qcms
     // FIXME: Remove these when we are done comparing with qcms.
     friend class DM::ColorCodecSrc;
     friend class ColorCodecBench;
 
     friend class SkSampledCodec;
     friend class SkIcoCodec;
 };
 #endif // SkCodec_DEFINED