Initial picture shader implementation

include/core/SkShader.h

Index: include/core/SkShader.h
diff --git a/include/core/SkShader.h b/include/core/SkShader.h
index acff959949ba5114bf2393dd4c0c181b376ab0b5..b61fb9a470a9e6f3811634b4f57eb82fb6737702 100644
--- a/include/core/SkShader.h
+++ b/include/core/SkShader.h
@@ -16,6 +16,7 @@
 #include "SkPaint.h"
 
 class SkPath;
+class SkPicture;
 class GrContext;
 class GrEffectRef;
 
@@ -345,6 +346,15 @@ public:
     static SkShader* CreateBitmapShader(const SkBitmap& src,
                                         TileMode tmx, TileMode tmy);
 
+    /** Call this to create a new shader that will draw with the specified picture.
+     *
+     *  @param src  The picture to use inside the shader

[reed1, 2014/04/07 15:30:09]

// Do we ref() the argument?

[f(malita), 2014/04/07 15:45:29]

Done.

+     *  @param tmx  The tiling mode to use when sampling the bitmap in the x-direction.
+     *  @param tmy  The tiling mode to use when sampling the bitmap in the y-direction.
+     *  @return     Returns a new shader object. Note: this function never returns null.
+    */
+    static SkShader* CreatePictureShader(SkPicture& src, TileMode tmx, TileMode tmy);

[reed1, 2014/04/07 15:30:09]

I know drawPicture takes a &, but I think that is wacky. Everywhere else that we
ref() an arg we take a pointer. I think we should do that here as well.

[f(malita), 2014/04/07 15:45:29]

Done.

+
     SK_TO_STRING_VIRT()
     SK_DEFINE_FLATTENABLE_TYPE(SkShader)