allow pictures to have a full bounds

src/core/SkRecorder.cpp

 /*
  * Copyright 2014 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkData.h"
 #include "SkRecorder.h"
 #include "SkPatchUtils.h"
 #include "SkPicture.h"
 

[robertphillips, 2014/11/18 20:51:53]

Add one of these to SkRect?

[reed1, 2014/11/18 22:05:30]

Love to, but I have to clean up SkRect.h first, which *has* roundOut() that
doesn't return anything but affects the "this" inplace :(

[reed1, 2014/11/19 14:49:16]

Done.

-// SkCanvas will fail in mysterious ways if it doesn't know the real width and height.
+static SkIRect round_out(const SkRect& r) {
+    SkIRect ir;
+    r.roundOut(&ir);
+    return ir;
+}
+
 SkRecorder::SkRecorder(SkRecord* record, int width, int height)
-    : SkCanvas(width, height, SkCanvas::kConservativeRasterClip_InitFlag)
+    : SkCanvas(SkIRect::MakeWH(width, height), SkCanvas::kConservativeRasterClip_InitFlag)
     , fRecord(record)
     , fSaveLayerCount(0) {}
 
+SkRecorder::SkRecorder(SkRecord* record, const SkRect& bounds)
+    : SkCanvas(round_out(bounds), SkCanvas::kConservativeRasterClip_InitFlag)
+    , fRecord(record)
+    , fSaveLayerCount(0) {}
+
 SkRecorder::~SkRecorder() {
     fDrawableList.unrefAll();
 }
 
 void SkRecorder::forgetRecord() {
     fDrawableList.unrefAll();
     fDrawableList.reset();
     fRecord = NULL;
 }
 
@@ skipping 338 matching lines @@
     APPEND(EndCommentGroup);
 }
 
 bool SkRecorder::isDrawingToLayer() const {
     return fSaveLayerCount > 0;
 }
 
 void SkRecorder::drawData(const void* data, size_t length) {
     APPEND(DrawData, copy((const char*)data), length);
 }