Adding a PictureResolution option to SkPictureImageFilter

include/effects/SkPictureImageFilter.h

 /*
  * Copyright 2013 The Android Open Source Project
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef SkPictureImageFilter_DEFINED
 #define SkPictureImageFilter_DEFINED
 
 #include "SkImageFilter.h"
 #include "SkPicture.h"
 
 class SK_API SkPictureImageFilter : public SkImageFilter {
 public:
+    enum PictureResolution {
+        /** 
+         * In this mode, which is the default, the picture is reasterized in at
+         * a resolution that matches the device that filter is rendering to.
+         */
+        kDeviceSpace_PictureResolution,
+
+        /** 
+         * In this mode, the picture is rasterized at a resolution such that pixels have
+         * unit size in the local (pre-transform) coordinate space. This is equivalent
+         * to rendering in record-time device space and the resampling the result to
+         * pring in into the playback device space.
+         */
+        kLocalSpace_PictureResolution
+    };
+
     /**
      *  Refs the passed-in picture.
      */
-    static SkPictureImageFilter* Create(const SkPicture* picture, int32_t uniqueID = 0) {
-        return SkNEW_ARGS(SkPictureImageFilter, (picture, uniqueID));
+    static SkPictureImageFilter* Create(const SkPicture* picture, int32_t uniqueID = 0,

[reed2, 2014/11/28 15:33:52]

api nit: can we not have so many default arguments?

same comment for the next factory.

+                                        PictureResolution res = kDeviceSpace_PictureResolution,
+                                        SkPaint::FilterLevel level = SkPaint::kLow_FilterLevel) {
+        return SkNEW_ARGS(SkPictureImageFilter, (picture, uniqueID, res, level));
     }
 
     /**
      *  Refs the passed-in picture. cropRect can be used to crop or expand the destination rect when
      *  the picture is drawn. (No scaling is implied by the dest rect; only the CTM is applied.)
      */
-    static SkPictureImageFilter* Create(const SkPicture* picture, const SkRect& cropRect, uint32_t uniqueID = 0) {
-        return SkNEW_ARGS(SkPictureImageFilter, (picture, cropRect, uniqueID));
+    static SkPictureImageFilter* Create(const SkPicture* picture, const SkRect& cropRect,
+                                        uint32_t uniqueID = 0,
+                                        PictureResolution res = kDeviceSpace_PictureResolution,
+                                        SkPaint::FilterLevel level = SkPaint::kLow_FilterLevel) {
+        return SkNEW_ARGS(SkPictureImageFilter, (picture, cropRect, uniqueID, res, level));
     }
 
     SK_DECLARE_PUBLIC_FLATTENABLE_DESERIALIZATION_PROCS(SkPictureImageFilter)
 
 protected:
-    explicit SkPictureImageFilter(const SkPicture* picture, uint32_t uniqueID);
-    SkPictureImageFilter(const SkPicture* picture, const SkRect& cropRect, uint32_t uniqueID);
+    explicit SkPictureImageFilter(const SkPicture* picture, uint32_t uniqueID, PictureResolution,
+                                  SkPaint::FilterLevel);
+    SkPictureImageFilter(const SkPicture* picture, const SkRect& cropRect, uint32_t uniqueID,
+                         PictureResolution, SkPaint::FilterLevel);
     virtual ~SkPictureImageFilter();
     /*  Constructs an SkPictureImageFilter object from an SkReadBuffer.
      *  Note: If the SkPictureImageFilter object construction requires bitmap
      *  decoding, the decoder must be set on the SkReadBuffer parameter by calling
      *  SkReadBuffer::setBitmapDecoder() before calling this constructor.
      *  @param SkReadBuffer Serialized picture data.
      */
 #ifdef SK_SUPPORT_LEGACY_DEEPFLATTENING
     explicit SkPictureImageFilter(SkReadBuffer&);
 #endif
     virtual void flatten(SkWriteBuffer&) const SK_OVERRIDE;
     virtual bool onFilterImage(Proxy*, const SkBitmap& src, const Context&,
                                SkBitmap* result, SkIPoint* offset) const SK_OVERRIDE;
 
 private:
-    const SkPicture* fPicture;
-    SkRect           fCropRect;
+    void drawPictureAtNativeResolution(Proxy*, SkBaseDevice*, const SkIRect& deviceBounds,
+                                       const Context&) const;
+    void drawPictureAtOriginalResolution(Proxy*, SkBaseDevice*, const SkIRect& deviceBounds,
+                                       const Context&) const;
+
+    const SkPicture*      fPicture;
+    SkRect                fCropRect;
+    SkPaint::FilterLevel  fFilterLevel;
+    PictureResolution     fPictureResolution;
     typedef SkImageFilter INHERITED;
 };
 
 #endif