shrink our 'largest' so that its non-empty

Bug: skia:
Change-Id: Ib12fd9491069440c85d5aa9c9d6d26787f03ef0d
Reviewed-on: https://skia-review.googlesource.com/92643
Reviewed-by: Yuqian Li <liyuqian@google.com>
Commit-Queue: Yuqian Li <liyuqian@google.com>

1 files changed, 11 insertions, 11 deletions

diff --git a/src/core/SkRectPriv.h b/src/core/SkRectPriv.h
index 8b663ab3dd..a8f40db005 100644
--- a/src/core/SkRectPriv.h
+++ b/src/core/SkRectPriv.h
@@ -12,29 +12,29 @@
 
 class SkRectPriv {
 public:
-    static SkRect MakeLargestS32() {
+    static SkIRect MakeILargest() {
         const int32_t ihalf = SK_MaxS32 >> 1;
-        const SkScalar half = SkIntToScalar(ihalf);
+        return { -ihalf, -ihalf, ihalf, ihalf };
+    }
 
-        return { -half, -half, half, half };
+    static SkIRect MakeILargestInverted() {
+        return { SK_MaxS32, SK_MaxS32, SK_MinS32, SK_MinS32 };
     }
 
-    static SkRect MakeLargest() {
-        return { SK_ScalarMin, SK_ScalarMin, SK_ScalarMax, SK_ScalarMax };
+    static SkRect MakeLargestS32() {
+        SkRect r;
+        r.set(MakeILargest());
+        return r;
     }
 
-    static SkIRect MakeILargest() {
-        return { SK_MinS32, SK_MinS32, SK_MaxS32, SK_MaxS32 };
+    static SkRect MakeLargest() {
+        return { SK_ScalarMin, SK_ScalarMin, SK_ScalarMax, SK_ScalarMax };
     }
 
     static SkRect MakeLargestInverted() {
         return { SK_ScalarMax, SK_ScalarMax, SK_ScalarMin, SK_ScalarMin };
     }
 
-    static SkIRect MakeILargestInverted() {
-        return { SK_MaxS32, SK_MaxS32, SK_MinS32, SK_MinS32 };
-    }
-
     static void GrowToInclude(SkRect* r, const SkPoint& pt) {
         r->fLeft  =  SkMinScalar(pt.fX, r->fLeft);
         r->fRight =  SkMaxScalar(pt.fX, r->fRight);