Allow reset of the SkPictureRecorder cull rect before endRecording.

include/core/SkPictureRecorder.h

 /*
  * Copyright 2014 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef SkPictureRecorder_DEFINED
 #define SkPictureRecorder_DEFINED
 
@@ skipping 57 matching lines @@
      *  must call unref() when they are done using it.
      *
      *  The returned picture is immutable. If during recording drawables were added to the canvas,
      *  these will have been "drawn" into a recording canvas, so that this resulting picture will
      *  reflect their current state, but will not contain a live reference to the drawables
      *  themselves.
      */
     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.
      *
      *  Unlike endRecordingAsPicture(), which returns an immutable picture, the returned drawable
      *  may contain live references to other drawables (if they were added to the recording canvas)
      *  and therefore this drawable will reflect the current state of those nested drawables anytime
      *  it is drawn or a new picture is snapped from it (by calling drawable->newPictureSnapshot()).
      */
     SkDrawable* SK_WARN_UNUSED_RESULT endRecordingAsDrawable();
@@ skipping 18 matching lines @@
     SkRect                        fCullRect;
     SkAutoTUnref<SkBBoxHierarchy> fBBH;
     SkAutoTUnref<SkRecorder>      fRecorder;
     SkAutoTUnref<SkRecord>        fRecord;
     SkMiniRecorder                fMiniRecorder;
 
     typedef SkNoncopyable INHERITED;
 };
 
 #endif