SkScaledCodec class

include/codec/SkCodec.h

Index: include/codec/SkCodec.h
diff --git a/include/codec/SkCodec.h b/include/codec/SkCodec.h
index 1cdc88d4adcfc0dc4e028913348dd21c287e50e7..d8ea2e1430d6fdb0c7c773fad00e9167cd887f93 100644
--- a/include/codec/SkCodec.h
+++ b/include/codec/SkCodec.h
@@ -149,6 +149,7 @@ public:
         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;
@@ -161,6 +162,7 @@ public:
          *  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.

     };
 
     /**
@@ -244,6 +246,10 @@ public:
         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*);
 
@@ -288,6 +294,7 @@ protected:
     }
 
     virtual bool onReallyHasAlpha() const { return false; }
+    virtual bool onIsInterlaced() { return false; }
 
     enum RewindState {
         kRewound_RewindState,