allow pictures to have a full bounds

src/core/SkBBHFactory.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 "SkBBHFactory.h"
 #include "SkRTree.h"
 #include "SkTileGrid.h"
 
-
-SkBBoxHierarchy* SkRTreeFactory::operator()(int width, int height) const {
-    SkScalar aspectRatio = SkScalarDiv(SkIntToScalar(width), SkIntToScalar(height));
+SkBBoxHierarchy* SkRTreeFactory::operator()(const SkRect& bounds) const {
+    SkScalar aspectRatio = bounds.width() / bounds.height();
     return SkNEW_ARGS(SkRTree, (aspectRatio));
 }
 
-SkBBoxHierarchy* SkTileGridFactory::operator()(int width, int height) const {
+SkBBoxHierarchy* SkTileGridFactory::operator()(const SkRect& bounds) const {
     SkASSERT(fInfo.fMargin.width() >= 0);
     SkASSERT(fInfo.fMargin.height() >= 0);
+
+    const int width = SkScalarRoundToInt(bounds.width());

[mtklein, 2014/11/18 21:06:49]

Is RoundToInt away-from-zero?

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

Nope, but I can change it to that.
Really what we probably want is to roundOut the rect, and that that guy's
width/height.

[mtklein, 2014/11/18 22:26:11]

That sounds right.

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

Done.

+    const int height = SkScalarRoundToInt(bounds.height());
+
     // Note: SkIRects are non-inclusive of the right() column and bottom() row.
     // For example, an SkIRect at 0,0 with a size of (1,1) will only have
     // content at pixel (0,0) and will report left=0 and right=1, hence the
     // "-1"s below.
     int xTileCount = (width + fInfo.fTileInterval.width() - 1) / fInfo.fTileInterval.width();
     int yTileCount = (height + fInfo.fTileInterval.height() - 1) / fInfo.fTileInterval.height();
     return SkNEW_ARGS(SkTileGrid, (xTileCount, yTileCount, fInfo));
 }