fix fuzz test that exceeds numeric limit

fix fuzz test that exceeds numeric limit

The extreme values here exceed an internal test that expects
computed numbers to be less than MAX_FLT. Use MAX_DBL instead.

R=mmoroz@chromium.org
TBR=reed@google.com
BUG=624346
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2103903004

Committed: https://skia.googlesource.com/skia/+/343382e3acc8369f7bd4328e7c807255b5776fe5

[caryclark, 2016-06-29 14:40:18]

Description was changed from

==========
fix fuzz test that exceeds numeric limit

The extreme values here exceed an internal test that expects
computed numbers to be less than MAX_FLT. Use MAX_DBL instead.

R=mmoroz@chromium.org
TBR=reed@google.com
BUG=624346
==========

to

==========
fix fuzz test that exceeds numeric limit

The extreme values here exceed an internal test that expects
computed numbers to be less than MAX_FLT. Use MAX_DBL instead.

R=mmoroz@chromium.org
TBR=reed@google.com
BUG=624346
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2103903004
==========

[caryclark, 2016-06-29 14:41:31]

The CQ bit was checked by caryclark@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-06-29 14:41:39]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[mmoroz, 2016-06-29 14:52:34]

mmoroz@chromium.org changed reviewers:
+ mbarbella@chromium.org

[mmoroz, 2016-06-29 14:52:35]

LGTM

Thought I'm not too experienced with Skia, adding mbarbella@ just a fyi.

https://codereview.chromium.org/2103903004/diff/1/src/pathops/SkOpSegment.cpp
File src/pathops/SkOpSegment.cpp (right):

https://codereview.chromium.org/2103903004/diff/1/src/pathops/SkOpSegment.cpp...
src/pathops/SkOpSegment.cpp:1134: ? foundSpan->upCastable() ?
foundSpan->upCast()->next() : nullptr
Not sure if it is a good idea to use a nested ternary conditional operators. For
me looks not too readable and line length is exceeded by one symbol.

[commit-bot: I haz the power, 2016-06-29 15:13:08]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-29 15:13:08]

Dry run: This issue passed the CQ dry run.

[caryclark, 2016-06-29 15:17:30]

The CQ bit was checked by caryclark@google.com

[commit-bot: I haz the power, 2016-06-29 15:17:41]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-06-29 15:18:40]

Description was changed from

==========
fix fuzz test that exceeds numeric limit

The extreme values here exceed an internal test that expects
computed numbers to be less than MAX_FLT. Use MAX_DBL instead.

R=mmoroz@chromium.org
TBR=reed@google.com
BUG=624346
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2103903004
==========

to

==========
fix fuzz test that exceeds numeric limit

The extreme values here exceed an internal test that expects
computed numbers to be less than MAX_FLT. Use MAX_DBL instead.

R=mmoroz@chromium.org
TBR=reed@google.com
BUG=624346
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2103903004

Committed:
https://skia.googlesource.com/skia/+/343382e3acc8369f7bd4328e7c807255b5776fe5
==========

[commit-bot: I haz the power, 2016-06-29 15:18:42]

Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/343382e3acc8369f7bd4328e7c807255b5776fe5

[Martin Barbella, 2016-06-29 16:40:24]

LGTM. Thanks for the fix.