Deleted SkImage::newShaderClamp: not used, not implemented.

src/image/SkImage_Codec.cpp

Index: src/image/SkImage_Codec.cpp
diff --git a/src/image/SkImage_Codec.cpp b/src/image/SkImage_Codec.cpp
index 3d815ce7f437409156b94c009a26e733ca77a270..5c543069526977cd40376a2ca13459837fddf12a 100644
--- a/src/image/SkImage_Codec.cpp
+++ b/src/image/SkImage_Codec.cpp
@@ -21,6 +21,7 @@ public:
     virtual void onDraw(SkCanvas*, SkScalar, SkScalar, const SkPaint*) SK_OVERRIDE;
     virtual void onDrawRectToRect(SkCanvas*, const SkRect*, const SkRect&, const SkPaint*) SK_OVERRIDE;
 
+    virtual SkShader* onNewShader(SkShader::TileMode, SkShader::TileMode) const SK_OVERRIDE;
 private:
     SkData*     fEncodedData;
     SkBitmap    fBitmap;
@@ -39,6 +40,10 @@ SkImage_Codec::~SkImage_Codec() {
     fEncodedData->unref();
 }
 
+SkShader* SkImage_Codec::onNewShader(SkShader::TileMode tileX, SkShader::TileMode tileY) const {

[Justin Novosad, 2014/07/10 19:24:36]

Do you have test coverage for this guy?

[Rémi Piotaix, 2014/07/10 20:16:04]

It seems that SkImage_Codec is never used...
Method onNewShader removed from SkImage_Codec + default implementation returns
NULL.

+    return SkShader::CreateBitmapShader(fBitmap, tileX, tileY, NULL);
+}
+
 void SkImage_Codec::onDraw(SkCanvas* canvas, SkScalar x, SkScalar y, const SkPaint* paint) {
     if (!fBitmap.pixelRef()) {
         if (!SkImageDecoder::DecodeMemory(fEncodedData->bytes(), fEncodedData->size(), &fBitmap)) {