Allow reset of the SkPictureRecorder cull rect before endRecording.

include/core/SkPictureRecorder.h

Index: include/core/SkPictureRecorder.h
diff --git a/include/core/SkPictureRecorder.h b/include/core/SkPictureRecorder.h
index 811d02a36e141f6eff06a23cafc8d1f5fa7747dc..07dbe5f90b693248906af3960c98d71fc21e84bf 100644
--- a/include/core/SkPictureRecorder.h
+++ b/include/core/SkPictureRecorder.h
@@ -75,6 +75,20 @@ public:
     SkPicture* SK_WARN_UNUSED_RESULT endRecordingAsPicture();
 
     /**
+     *  Signal that the caller is done recording, and update the parameters to use for bounding
+     *  box hierarchy (BBH) generation. The behavior is the same as calling
+     *  endRecordingAsPicture(), except that this method updates the parameters initially passed
+     *  into beginRecording.
+     *  @param cullRect the new culling rectangle to use as the overall bound for BBH generation
+     *                  and subsequent culling operations.
+     *  @param bbhFactory factory to create desired acceleration structure. Any existing factory
+     *                     is cleared.
+     *  @return the picture containing the recorded content.
+     */
+    SkPicture* SK_WARN_UNUSED_RESULT endRecordingAsPicture(const SkRect& cullRect,
+                                                           SkBBHFactory* bbhFactory = NULL);

[mtklein, 2015/07/07 19:18:16]

Let's kill off bbhFactory too?

The factory and cull rect being out of sync is something that's only technically
a problem, but in an M45-M46 time frame we'll always be using RTree where it
won't break anything, and is unlikely to have any effect at all (that aspect
ratio thing is very minor).

+
+    /**
      *  Signal that the caller is done recording. This invalidates the canvas returned by
      *  beginRecording/getRecordingCanvas. Ownership of the object is passed to the caller, who
      *  must call unref() when they are done using it.