Move all knowledge of X sampling into SkScaledCodec

include/codec/SkScaledCodec.h

Index: include/codec/SkScaledCodec.h
diff --git a/include/codec/SkScaledCodec.h b/include/codec/SkScaledCodec.h
index 20428d8d73f40b0a391dfa061131fe1358fbb27c..92ef19e74f702810b25111a290cff999aa2eb0e4 100644
--- a/include/codec/SkScaledCodec.h
+++ b/include/codec/SkScaledCodec.h
@@ -22,23 +22,7 @@ public:
 
     virtual ~SkScaledCodec();
 
-    /**
-     * returns whether a destination's dimensions are supported for down sampling
-     */
-    static bool DimensionsSupportedForSampling(const SkImageInfo& srcInfo, 
-                                               const SkImageInfo& dstInfo) {
-        // heights must be equal as no native y sampling is supported
-        if (dstInfo.height() != srcInfo.height()) {
-            return false;
-        }
-        // only support down sampling, dstWidth cannot be larger that srcWidth
-        if(dstInfo.width() > srcInfo.width()) {
-            return false;
-        }
-        return true;   
-    }
-
-    static void ComputeSampleSize(const SkImageInfo& dstInfo, const SkImageInfo& srcInfo,
+    static void ComputeSampleSize(const SkISize& dstDim, const SkISize& srcDim,

[msarett, 2015/10/01 19:34:33]

+1 for this change.  I have a CL somewhere that makes this same change as
well...

[scroggo, 2015/10/01 21:16:57]

Acknowledged.

                                   int* sampleSizeX, int* sampleSizeY);
 
 protected:
@@ -46,6 +30,7 @@ protected:
      * Recommend a set of destination dimensions given a requested scale
      */
     SkISize onGetScaledDimensions(float desiredScale) const override;
+    bool onDimensionsSupported(const SkISize&) override;
 
     Result onGetPixels(const SkImageInfo&, void*, size_t, const Options&, SkPMColor*, int*)
             override;