summaryrefslogtreecommitdiff
path: root/core
diff options
context:
space:
mode:
authorCary Clark <caryclark@google.com>2017-04-12 15:17:14 -0400
committerChromium commit bot <commit-bot@chromium.org>2017-04-12 19:29:46 +0000
commit94c34c9934bf93fc68d0ab0bb0c6696ce71b6441 (patch)
tree4be1f598815377a942735682fe98a7d63c0f27af /core
parentdeadbfb7874ee8627d3a27efcfb965233f0c5cd4 (diff)
downloadpdfium-94c34c9934bf93fc68d0ab0bb0c6696ce71b6441.tar.xz
No gradient to draw if clip is empty
If a clip with bounds (0, 0, 0, 0) is used to specify the limits of a gradient, the clipping rectangle will be degenerate. Leave the clip empty in this case. R=dsinclair@chromium.org Bug: 705736 Change-Id: I4a5845459c1527f2c643563d6bef4d6dcc6781d3 Reviewed-on: https://pdfium-review.googlesource.com/4071 Reviewed-by: dsinclair <dsinclair@chromium.org> Commit-Queue: Cary Clark <caryclark@google.com>
Diffstat (limited to 'core')
-rw-r--r--core/fxge/skia/fx_skia_device.cpp4
1 files changed, 3 insertions, 1 deletions
diff --git a/core/fxge/skia/fx_skia_device.cpp b/core/fxge/skia/fx_skia_device.cpp
index eacfd69c93..5f117cb4d6 100644
--- a/core/fxge/skia/fx_skia_device.cpp
+++ b/core/fxge/skia/fx_skia_device.cpp
@@ -522,7 +522,9 @@ void ClipAngledGradient(const SkPoint pts[2],
}
}
ASSERT(minBounds >= 0);
- ASSERT(maxBounds != minBounds && maxBounds >= 0);
+ ASSERT(maxBounds >= 0);
+ if (minBounds == maxBounds)
+ return;
// construct a clip parallel to the gradient that goes through
// rectPts[minBounds] and rectPts[maxBounds] and perpendicular to the
// gradient that goes through startEdgePt, endEdgePt.