[WIP] Add Context to SkDrawLooper.

include/core/SkDrawLooper.h

 
 /*
  * Copyright 2011 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 SkDrawLooper_DEFINED
@@ skipping 12 matching lines @@
     be called, allowing it to modify the canvas and/or paint for that draw call.
     More than that, via the next() method, the looper can modify the draw to be
     invoked multiple times (hence the name loop-er), allow it to perform effects
     like shadows or frame/fills, that require more than one pass.
 */
 class SK_API SkDrawLooper : public SkFlattenable {
 public:
     SK_DECLARE_INST_COUNT(SkDrawLooper)
 
     /**
+     *  Holds state during a draw. A new object is returned by init() and must
+     *  be passed into all subsequent calls of next(). The final call to next()
+     *  will also delete the context.
+     *  Subclasses of SkDrawLooper should create a subclass of this object to
+     *  hold state specific to their subclass.
+     */
+    struct SK_API DrawContext {};
+
+    /**
      *  Called right before something is being drawn. This will be followed by
      *  calls to next() until next() returns false.
      */
-    virtual void init(SkCanvas*) = 0;
+    virtual DrawContext* init(SkCanvas*) const = 0;
 
     /**
      *  Called in a loop (after init()). Each time true is returned, the object
      *  is drawn (possibly with a modified canvas and/or paint). When false is
      *  finally returned, drawing for the object stops.
      *
      *  On each call, the paint will be in its original state, but the canvas
      *  will be as it was following the previous call to next() or init().
      *
      *  The implementation must ensure that, when next() finally returns false,
      *  that the canvas has been restored to the state it was initially, before
      *  init() was first called.
      */
-    virtual bool next(SkCanvas*, SkPaint* paint) = 0;
+    bool next(SkCanvas* canvas, SkPaint* paint, DrawContext* context) const;

[scroggo, 2014/02/06 17:57:13]

It's a little weird that the parameter that's passed in may be deleted, but it
may be the right way to do it. Could you add a comment here explaining that.
(Assuming it is still dynamically allocated. Also, I know there's a comment
above DrawContext, but I think it should be here as well.)

 
     /**
      * The fast bounds functions are used to enable the paint to be culled early
      * in the drawing pipeline. If a subclass can support this feature it must
      * return true for the canComputeFastBounds() function.  If that function
      * returns false then computeFastBounds behavior is undefined otherwise it
      * is expected to have the following behavior. Given the parent paint and
      * the parent's bounding rect the subclass must fill in and return the
      * storage rect, where the storage rect is with the union of the src rect
      * and the looper's bounding rect.
      */
     virtual bool canComputeFastBounds(const SkPaint& paint);
     virtual void computeFastBounds(const SkPaint& paint,
                                    const SkRect& src, SkRect* dst);
 
     SkDEVCODE(virtual void toString(SkString* str) const = 0;)
     SK_DEFINE_FLATTENABLE_TYPE(SkDrawLooper)
 
 protected:
     SkDrawLooper() {}
     SkDrawLooper(SkReadBuffer& buffer) : INHERITED(buffer) {}
 
+    virtual bool next_internal(

[scroggo, 2014/02/06 17:57:13]

I think our typical naming convention would be onNext().

+        SkCanvas* canvas, SkPaint* paint, DrawContext* context) const = 0;
+
 private:
     typedef SkFlattenable INHERITED;
 };
 
 #endif