diff options
author | egdaniel <egdaniel@google.com> | 2014-11-05 05:02:27 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2014-11-05 05:02:27 -0800 |
commit | 21402e3851454e90329bad52893f29959da086f7 (patch) | |
tree | 06923683a74242ca885cf1c667974ea7fd1260db /src/utils/SkDashPath.cpp | |
parent | 9f2251c73ed6f417dd1057d487bf523e04488440 (diff) |
Bug fix for cull_line in SkDashPathEffect.
In cull_line we must also check if both points are the same. Otherwise we
fail the assert in the else "SkASSERT(dy && !dx)".
This is currently blocking the roll as it fails a webkit test.
BUG=skia:
Review URL: https://codereview.chromium.org/703783002
Diffstat (limited to 'src/utils/SkDashPath.cpp')
-rw-r--r-- | src/utils/SkDashPath.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/utils/SkDashPath.cpp b/src/utils/SkDashPath.cpp index 02de98f1d6..4b2b33d2c4 100644 --- a/src/utils/SkDashPath.cpp +++ b/src/utils/SkDashPath.cpp @@ -119,7 +119,7 @@ static bool cull_path(const SkPath& srcPath, const SkStrokeRec& rec, SkTSwap(minX, maxX); } - SkASSERT(minX < maxX); + SkASSERT(minX <= maxX); if (maxX < bounds.fLeft || minX > bounds.fRight) { return false; } |