SkScaledCodec class

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 131 matching lines @@
         kNo_ZeroInitialized,
     };
 
     /**
      *  Additional options to pass to getPixels.
      */
     struct Options {
         Options()
             : fZeroInitialized(kNo_ZeroInitialized)
             , fSubset(NULL)
+            , fScaled(0)

[scroggo, 2015/07/27 15:09:19]

nit: false

[emmaleer, 2015/07/27 18:31:38]

Acknowledged.

         {}
 
         ZeroInitialized fZeroInitialized;
         /**
          *  If not NULL, represents a subset of the original image to decode.
          *
          *  Must be within the bounds returned by getInfo().
          *
          *  If the EncodedFormat is kWEBP_SkEncodedFormat (the only one which
          *  currently supports subsets), the top and left values must be even.
          */
         SkIRect*        fSubset;
+        bool            fScaled;

[scroggo, 2015/07/27 15:09:19]

please add comments explaining how this is used (and its default value)

[emmaleer, 2015/07/27 18:31:38]

Acknowledged.

     };
 
     /**
      *  Decode into the given pixels, a block of memory of size at
      *  least (info.fHeight - 1) * rowBytes + (info.fWidth *
      *  bytesPerPixel)
      *
      *  Repeated calls to this function should give the same results,
      *  allowing the PixelRef to be immutable.
      *
@@ skipping 63 matching lines @@
      *  of the encoded data, but then never use any colors which have alpha
      *  less than 100%. This function can be called *after* decoding to
      *  determine if such an image truly had alpha. Calling it before decoding
      *  is undefined.
      *  FIXME: see skbug.com/3582.
      */
     bool reallyHasAlpha() const {
         return this->onReallyHasAlpha();
     }
 
+    bool isInterlaced() {

[scroggo, 2015/07/27 15:09:19]

Please add comments, here and on the virtual method.

[emmaleer, 2015/07/27 18:31:38]

Acknowledged.

+        return this->onIsInterlaced();
+    }
+
 protected:
     SkCodec(const SkImageInfo&, SkStream*);
 
     virtual SkISize onGetScaledDimensions(float /* desiredScale */) const {
         // By default, scaling is not supported.
         return this->getInfo().dimensions();
     }
 
     virtual SkEncodedFormat onGetEncodedFormat() const = 0;
 
@@ skipping 24 matching lines @@
      *      responsible for deleting the returned object.
      */
     virtual SkScanlineDecoder* onGetScanlineDecoder(const SkImageInfo& dstInfo,
                                                     const Options& options,
                                                     SkPMColor ctable[],
                                                     int* ctableCount) {
         return NULL;
     }
 
     virtual bool onReallyHasAlpha() const { return false; }
+    virtual bool onIsInterlaced() { return false; }
 
     enum RewindState {
         kRewound_RewindState,
         kNoRewindNecessary_RewindState,
         kCouldNotRewind_RewindState
     };
     /**
      *  If the stream was previously read, attempt to rewind.
      *  @returns:
      *      kRewound if the stream needed to be rewound, and the
@@ skipping 15 matching lines @@
     SkStream* stream() {
         return fStream.get();
     }
 
 private:
     const SkImageInfo       fInfo;
     SkAutoTDelete<SkStream> fStream;
     bool                    fNeedsRewind;
 };
 #endif // SkCodec_DEFINED