check for irect with overflow width/height

Bug:798066
Change-Id: Iac324ac5a32fae241a528751c84279ce60ac4baf
Reviewed-on: https://skia-review.googlesource.com/90544
Reviewed-by: Mike Klein <mtklein@google.com>
Commit-Queue: Mike Reed <reed@google.com>

1 files changed, 24 insertions, 0 deletions

diff --git a/src/core/SkRectPriv.h b/src/core/SkRectPriv.h
new file mode 100644
index 0000000000..4dcc4c56ce
--- /dev/null
+++ b/src/core/SkRectPriv.h
@@ -0,0 +1,24 @@
+/*
+ * Copyright 2018 Google Inc.
+ *
+ * Use of this source code is governed by a BSD-style license that can be
+ * found in the LICENSE file.
+ */
+
+#ifndef SkRectPriv_DEFINED
+#define SkRectPriv_DEFINED
+
+#include "SkRect.h"
+
+class SkRectPriv {
+public:
+    // Returns true iff width and height are positive. Catches inverted, empty, and overflowing
+    // (way too big) rects. This is used by clients that want a non-empty rect that they can also
+    // actually use its computed width/height.
+    //
+    static bool PositiveDimensions(const SkIRect& r) {
+        return r.width() > 0 && r.height() > 0;
+    }
+};
+
+#endif