commit e191ce68f00ff4b517fdf244b3311904858a25ec
author Brian Osman <brianosman@google.com> Thu Sep 14 15:43:50 2017 -0400
committer Skia Commit-Bot <skia-commit-bot@chromium.org> Thu Sep 14 20:39:55 2017 +0000
tree e40cf1c6929d5f9f7d1dbd7549775d90469149c3
parent f1748f5a9238a0d3e189d50fc5e57ae8b8ec087c

Remove unnecessary code in SkGpuDevice::drawRRect

Bug: skia:
Change-Id: Ied1e4bd89305695957faa2328e363304beb90a21
Reviewed-on: https://skia-review.googlesource.com/46840
Reviewed-by: Brian Salomon <bsalomon@google.com>
Commit-Queue: Brian Osman <brianosman@google.com>

src/gpu/SkGpuDevice.cpp

diff --git a/src/gpu/SkGpuDevice.cpp b/src/gpu/SkGpuDevice.cpp
index 8e05d8a..3f0bdce 100644
--- a/src/gpu/SkGpuDevice.cpp
+++ b/src/gpu/SkGpuDevice.cpp
@@ -403,15 +403,7 @@
         SkRRect devRRect;
         if (rrect.transform(this->ctm(), &devRRect)) {
             if (devRRect.allCornersCircular()) {
-                SkRect maskRect;
-                if (mf->canFilterMaskGPU(devRRect, this->devClipBounds(),
-                                         this->ctm(), &maskRect)) {
-                    SkIRect finalIRect;
-                    maskRect.roundOut(&finalIRect);
-
-                    // we used to test finalIRect for quickReject, but that seems unlikely
-                    // given that the original shape was not rejected...
-
+                if (mf->canFilterMaskGPU(devRRect, this->devClipBounds(), this->ctm(), nullptr)) {
                     if (mf->directFilterRRectMaskGPU(this->context(), fRenderTargetContext.get(),
                                                      std::move(grPaint), this->clip(), this->ctm(),
                                                      style.strokeRec(), rrect, devRRect)) {