Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(323)

Issue 2209303004: make SpecialLineRec::addSegment assert more permissive to accomodate rounding error (Closed)

Created:
4 years, 4 months ago by lsalzman1
Modified:
4 years, 4 months ago
Reviewers:
caryclark
CC:
reviews_skia.org
Base URL:
https://skia.googlesource.com/skia@master
Target Ref:
refs/heads/master
Project:
skia
Visibility:
Public.

Description

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 Committed: https://skia.googlesource.com/skia/+/0adbd3e0a61c039fea1d47c006daa462bae42318

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M src/utils/SkDashPath.cpp View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 9 (4 generated)
lsalzman1
4 years, 4 months ago (2016-08-04 17:09:07 UTC) #3
caryclark
On 2016/08/04 17:09:07, lsalzman1 wrote: My apologies for not getting to this sooner. I was ...
4 years, 4 months ago (2016-08-08 19:56:42 UTC) #4
caryclark
lgtm
4 years, 4 months ago (2016-08-08 20:05:56 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2209303004/1
4 years, 4 months ago (2016-08-08 20:09:49 UTC) #7
commit-bot: I haz the power
4 years, 4 months ago (2016-08-08 20:40:31 UTC) #9
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/0adbd3e0a61c039fea1d47c006daa462bae42318

Powered by Google App Engine
This is Rietveld 408576698