change roundOut() to take a dst, so we can use that signature to return IRect

include/core/SkRect.h

 /*
  * Copyright 2006 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 SkRect_DEFINED
 #define SkRect_DEFINED
 
@@ skipping 788 matching lines @@
     void roundOut(SkIRect* dst) const {
         SkASSERT(dst);
         dst->set(SkScalarFloorToInt(fLeft), SkScalarFloorToInt(fTop),
                  SkScalarCeilToInt(fRight), SkScalarCeilToInt(fBottom));
     }
 
     /**
      *  Expand this rectangle by rounding its coordinates "out", choosing the
      *  floor of top and left, and the ceil of right and bottom. If this rect
      *  is already on integer coordinates, then it will be unchanged.
+     *
+     *  It is safe for this == dst
      */
-    void roundOut() {
-        this->set(SkScalarFloorToScalar(fLeft),
-                  SkScalarFloorToScalar(fTop),
-                  SkScalarCeilToScalar(fRight),
-                  SkScalarCeilToScalar(fBottom));
+    void roundOut(SkRect* dst) const {
+        dst->set(SkScalarFloorToScalar(fLeft),
+                 SkScalarFloorToScalar(fTop),
+                 SkScalarCeilToScalar(fRight),
+                 SkScalarCeilToScalar(fBottom));
     }
 
     /**
      *  Set the dst rectangle by rounding "in" this rectangle, choosing the
      *  ceil of top and left, and the floor of right and bottom. This does *not*
      *  call sort(), so it is possible that the resulting rect is inverted...
      *  e.g. left >= right or top >= bottom. Call isEmpty() to detect that.
      */
     void roundIn(SkIRect* dst) const {
         SkASSERT(dst);
@@ skipping 38 matching lines @@
      * rely on the existence of this function or the formatting of its output.
      */
     void dump() const {
         SkDebugf("{ l: %f, t: %f, r: %f, b: %f }", fLeft, fTop, fRight, fBottom);
     }
 #endif
 
 };
 
 #endif