diff options
author | lsalzman <lsalzman@mozilla.com> | 2016-08-08 13:40:27 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-08-08 13:40:27 -0700 |
commit | 0adbd3e0a61c039fea1d47c006daa462bae42318 (patch) | |
tree | 3e0c5998078176ab1d7da957c8a792f8470ae41e /src | |
parent | c97796b47b2f3e0ca8c04073c1576db21b58eedb (diff) |
make SpecialLineRec::addSegment assert more permissive to accomodate rounding error
Because the driving code that calls SpecialLineRec::addSegment uses double precision,
when converted back to single precision this may round such that d0 and d1 are the same.
This causes the assert to trigger. The simplest thing to do here is to modify the assert
to not be so picky and allow the rare/occasional equality.
Downstream Firefox bug report: https://bugzilla.mozilla.org/show_bug.cgi?id=1290628
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2209303004
Review-Url: https://codereview.chromium.org/2209303004
Diffstat (limited to 'src')
-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 e06c2d2da3..c0cdcc195d 100644 --- a/src/utils/SkDashPath.cpp +++ b/src/utils/SkDashPath.cpp @@ -186,7 +186,7 @@ public: } void addSegment(SkScalar d0, SkScalar d1, SkPath* path) const { - SkASSERT(d0 < fPathLength); + SkASSERT(d0 <= fPathLength); // clamp the segment to our length if (d1 > fPathLength) { d1 = fPathLength; |