Move SkShader::fLocalMatrix into SkShader constructor.

src/effects/SkRectShaderImageFilter.cpp

 /*
  * Copyright 2013 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkRectShaderImageFilter.h"
 #include "SkBitmap.h"
 #include "SkCanvas.h"
@@ skipping 49 matching lines @@
         return false;
     }
 
     SkAutoTUnref<SkBaseDevice> device(proxy->createDevice(bounds.width(),
                                                           bounds.height()));
     if (NULL == device.get()) {
         return false;
     }
     SkCanvas canvas(device.get());
     SkPaint paint;
-    paint.setShader(fShader);
     SkMatrix matrix(ctx.ctm());
     matrix.postTranslate(SkIntToScalar(-bounds.left()), SkIntToScalar(-bounds.top()));
-    fShader->setLocalMatrix(matrix);
+    paint.setShader(SkShader::CreateLocalMatrixWrapper(fShader, matrix));

[scroggo, 2014/04/24 17:02:53]

Don't we need to unref this shader? (Previously we were just passing fShader,
which we would unref later, but now we've created a new one.)

[Dominik Grewe, 2014/04/24 17:16:47]

Yeah, I think you're right.

     SkRect rect = SkRect::MakeWH(SkIntToScalar(bounds.width()), SkIntToScalar(bounds.height()));
     canvas.drawRect(rect, paint);
     *result = device.get()->accessBitmap(false);
     offset->fX = bounds.fLeft;
     offset->fY = bounds.fTop;
     return true;
 }