commit 3012cbaa20ce825e8bc9e7b31233bdbd87c68c33
author Mike Reed <reed@google.com> Mon Aug 26 10:31:13 2019 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Mon Aug 26 15:03:44 2019 +0000
tree 20a863c3cbd4081aa34a01019ef1bb933f88ba72
parent ff379423c1ede634b1fa35cf13140a436dd6e8a6

remove unneeded noemptycheck variants of intersect

The existing intersect logic already fails if either argument is empty,
without performing any extra checks.

Change-Id: I4cc4f1e63af7efbed4e1084284c1607c104ff361
Reviewed-on: https://skia-review.googlesource.com/c/skia/+/237142
Reviewed-by: Mike Reed <reed@google.com>
Auto-Submit: Mike Reed <reed@google.com>
Commit-Queue: Mike Reed <reed@google.com>

src/codec/SkWebpCodec.cpp

diff --git a/src/codec/SkWebpCodec.cpp b/src/codec/SkWebpCodec.cpp
index a66b2e5..20ded40 100644
--- a/src/codec/SkWebpCodec.cpp
+++ b/src/codec/SkWebpCodec.cpp
@@ -390,7 +390,7 @@
         SkASSERT(SkIsAlign2(subset.fLeft) && SkIsAlign2(subset.fTop));
         SkASSERT(this->getValidSubset(&subset) && subset == *options.fSubset);
 
-        if (!SkIRect::IntersectsNoEmptyCheck(subset, frameRect)) {
+        if (!SkIRect::Intersects(subset, frameRect)) {
             return kSuccess;
         }