Add nine patch type to SkRRect.

src/core/SkRRect.cpp

 /*
  * Copyright 2012 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkRRect.h"
 #include "SkMatrix.h"
 
@@ skipping 25 matching lines @@
     }
     fType = kSimple_Type;
     if (xRad >= SkScalarHalf(fRect.width()) && yRad >= SkScalarHalf(fRect.height())) {
         fType = kOval_Type;
         // TODO: assert that all the x&y radii are already W/2 & H/2
     }
 
     SkDEBUGCODE(this->validate();)
 }
 
+void SkRRect::setNinePatch(const SkRect& rect, SkScalar leftRad, SkScalar topRad,
+                           SkScalar rightRad, SkScalar bottomRad) {
+    if (rect.isEmpty()) {
+        this->setEmpty();
+        return;
+    }
+
+    leftRad = SkMaxScalar(leftRad, 0);
+    topRad = SkMaxScalar(topRad, 0);
+    rightRad = SkMaxScalar(rightRad, 0);
+    bottomRad = SkMaxScalar(bottomRad, 0);
+

[robertphillips, 2014/03/14 16:22:59]

I think we need to scale the x & y radii separately?

[bsalomon, 2014/03/14 16:30:20]

We don't in the other setters (which I just stole from).

[robertphillips, 2014/03/14 16:52:07]

Sigh - that is correct
(http://www.w3.org/TR/2010/WD-css3-background-20100612/#the-border-radius --
Overlapping Curves).

+    SkScalar scale = SK_Scalar1;
+    if (leftRad + rightRad > rect.width()) {
+        scale = SkScalarDiv(rect.width(), leftRad + rightRad);
+    }
+    if (topRad + bottomRad > rect.height()) {
+        scale = SkMinScalar(scale, SkScalarDiv(rect.width(), leftRad + rightRad));
+    }
+
+    if (scale < SK_Scalar1) {
+        leftRad = SkScalarMul(leftRad, scale);
+        topRad = SkScalarMul(topRad, scale);
+        rightRad = SkScalarMul(rightRad, scale);
+        bottomRad = SkScalarMul(bottomRad, scale);
+    }
+
+    if (leftRad == rightRad && topRad == bottomRad) {
+        if (leftRad >= SkScalarHalf(rect.width()) && topRad >= SkScalarHalf(rect.height())) {
+            fType = kOval_Type;

[robertphillips, 2014/03/14 16:22:59]

don't we also need "&& 0 == topRad"?

[bsalomon, 2014/03/14 16:30:20]

I don't think so... if the top radius is 0 then so is the bottom, which means
all the corners are square, right?

Maybe I should be jamming the them to 0 here so we store them that way?

[robertphillips, 2014/03/14 16:52:07]

You're right - probably both a comment and jamming them all to zero is in order

+        } else if (0 == leftRad) {
+            fType = kRect_Type;
+        } else {
+            fType = kSimple_Type;
+        }
+    } else {
+        fType = kNinePatch_Type;
+    }
+
+    fRect = rect;
+    fRadii[kUpperLeft_Corner].set(leftRad, topRad);
+    fRadii[kUpperRight_Corner].set(rightRad, topRad);
+    fRadii[kLowerRight_Corner].set(rightRad, bottomRad);
+    fRadii[kLowerLeft_Corner].set(leftRad, bottomRad);
+
+    SkDEBUGCODE(this->validate();)
+}
+
+
 void SkRRect::setRectRadii(const SkRect& rect, const SkVector radii[4]) {
     if (rect.isEmpty()) {
         this->setEmpty();
         return;
     }
 
     fRect = rect;
     memcpy(fRadii, radii, sizeof(fRadii));
 
     bool allCornersSquare = true;
@@ skipping 175 matching lines @@
     if (allRadiiEqual) {
         if (fRadii[0].fX >= SkScalarHalf(fRect.width()) &&
             fRadii[0].fY >= SkScalarHalf(fRect.height())) {
             fType = kOval_Type;
         } else {
             fType = kSimple_Type;
         }
         return;
     }
 
-    fType = kComplex_Type;
+    if (fRadii[kUpperLeft_Corner].fX == fRadii[kLowerLeft_Corner].fX &&
+        fRadii[kUpperLeft_Corner].fY == fRadii[kUpperRight_Corner].fY &&
+        fRadii[kUpperRight_Corner].fX == fRadii[kLowerRight_Corner].fX &&
+        fRadii[kLowerLeft_Corner].fY == fRadii[kLowerRight_Corner].fY) {
+        fType = kNinePatch_Type;
+    } else {
+        fType = kComplex_Type;
+    }
 }
 
 static bool matrix_only_scale_and_translate(const SkMatrix& matrix) {
     const SkMatrix::TypeMask m = (SkMatrix::TypeMask) (SkMatrix::kAffine_Mask
                                     | SkMatrix::kPerspective_Mask);
     return (matrix.getType() & m) == 0;
 }
 
 bool SkRRect::transform(const SkMatrix& matrix, SkRRect* dst) const {
     if (NULL == dst) {
@@ skipping 131 matching lines @@
             allRadiiZero = false;
         }
 
         if (fRadii[i].fX != fRadii[i-1].fX || fRadii[i].fY != fRadii[i-1].fY) {
             allRadiiSame = false;
         }
 
         if (0 != fRadii[i].fX && 0 != fRadii[i].fY) {
             allCornersSquare = false;
         }
     }

[robertphillips, 2014/03/14 16:22:59]

maybe a radii_make_nine_patch predicate to share code with above?

[bsalomon, 2014/03/14 16:30:20]

I'm not sure what you mean.

[robertphillips, 2014/03/14 16:52:07]

Add a helper function?

[bsalomon, 2014/03/14 17:15:31]

gotcha! will do. I thought you meant incorporating into the above loop.

+    bool patchesOfNine = fRadii[kUpperLeft_Corner].fX == fRadii[kLowerLeft_Corner].fX &&
+                         fRadii[kUpperLeft_Corner].fY == fRadii[kUpperRight_Corner].fY &&
+                         fRadii[kUpperRight_Corner].fX == fRadii[kLowerRight_Corner].fX &&
+                         fRadii[kLowerLeft_Corner].fY == fRadii[kLowerRight_Corner].fY;
 
     switch (fType) {
         case kEmpty_Type:
             SkASSERT(fRect.isEmpty());
             SkASSERT(allRadiiZero && allRadiiSame && allCornersSquare);
 
             SkASSERT(0 == fRect.fLeft && 0 == fRect.fTop &&
                      0 == fRect.fRight && 0 == fRect.fBottom);
             break;
         case kRect_Type:
             SkASSERT(!fRect.isEmpty());
             SkASSERT(allRadiiZero && allRadiiSame && allCornersSquare);
             break;
         case kOval_Type:
             SkASSERT(!fRect.isEmpty());
             SkASSERT(!allRadiiZero && allRadiiSame && !allCornersSquare);
 
             for (int i = 0; i < 4; ++i) {
                 SkASSERT(SkScalarNearlyEqual(fRadii[i].fX, SkScalarHalf(fRect.width())));
                 SkASSERT(SkScalarNearlyEqual(fRadii[i].fY, SkScalarHalf(fRect.height())));
             }
             break;
         case kSimple_Type:
             SkASSERT(!fRect.isEmpty());
             SkASSERT(!allRadiiZero && allRadiiSame && !allCornersSquare);
             break;
+        case kNinePatch_Type:
+            SkASSERT(!fRect.isEmpty());
+            SkASSERT(!allRadiiZero && !allRadiiSame && !allCornersSquare);
+            SkASSERT(patchesOfNine);
+            break;
         case kComplex_Type:
             SkASSERT(!fRect.isEmpty());
             SkASSERT(!allRadiiZero && !allRadiiSame && !allCornersSquare);
+            SkASSERT(!patchesOfNine);
             break;
         case kUnknown_Type:
             // no limits on this
             break;
     }
 }
 #endif // SK_DEBUG
 
 ///////////////////////////////////////////////////////////////////////////////