Initial error handling code

include/core/SkRect.h

Index: include/core/SkRect.h
diff --git a/include/core/SkRect.h b/include/core/SkRect.h
index d8919ae5d6b888d3195cc57ce7346bc905cf2a8c..bd5d026d5067de4fbf28e1c37ebaea8d518370a5 100644
--- a/include/core/SkRect.h
+++ b/include/core/SkRect.h
@@ -100,6 +100,8 @@ struct SK_API SkIRect {
      */
     bool isEmpty() const { return fLeft >= fRight || fTop >= fBottom; }
 
+    bool isInverted() const { return fLeft > fRight || fTop > fBottom; }

[mtklein, 2013/09/24 22:52:18]

Is there any performance lost by implementing this inline in SkValidationUtils? 
Not sure I see why this needs to be a new method if only used there.

[sugoi1, 2013/09/25 21:15:27]

It's just that I'm using this functionality in SkValidationUtils.h and in
SkImageFilter.cpp, so I didn't want to duplicate it. Also, the word "inverted"
is already used in this file with "setLargestInverted()", so it seemed to follow
the general nomenclature of SkRect.

+
     bool isLargest() const { return SK_MinS32 == fLeft &&
                                     SK_MinS32 == fTop &&
                                     SK_MaxS32 == fRight &&
@@ -419,6 +421,8 @@ struct SK_API SkRect {
      */
     bool isEmpty() const { return fLeft >= fRight || fTop >= fBottom; }
 
+    bool isInverted() const { return fLeft > fRight || fTop > fBottom; }
+
     /**
      *  Returns true iff all values in the rect are finite. If any are
      *  infinite or NaN (or SK_FixedNaN when SkScalar is fixed) then this