path ops -- handle non-finite numbers

path ops -- handle non-finite numbers

Op() and Simplify() do nothing if the input
is non-finite. Add code and tests.
Committed: https://code.google.com/p/skia/source/detail?r=8882

[commit-bot: I haz the power, 2013-04-26 16:39:08]

No reviewers yet.

[reed1, 2013-04-26 17:49:14]

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

https://codereview.chromium.org/14407006/diff/1/src/pathops/SkOpEdgeBuilder.c...
src/pathops/SkOpEdgeBuilder.cpp:167: if (!is_finite_point(pointsPtr[0])) {
Would it be faster to just ask for the path's bounds at the very beginning,
since that can be cached, and should report if the path was finite?

[caryclark, 2013-04-26 19:44:58]

path ops -- handle non-finite numbers

Op() and Simplify() do nothing if the input
is non-finite. Add code and tests.

[caryclark, 2013-04-26 19:47:30]

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

https://codereview.chromium.org/14407006/diff/1/src/pathops/SkOpEdgeBuilder.c...
src/pathops/SkOpEdgeBuilder.cpp:167: if (!is_finite_point(pointsPtr[0])) {
Yes indeed. Done and uploaded.

On 2013/04/26 17:49:14, reed1 wrote:
> Would it be faster to just ask for the path's bounds at the very beginning,
> since that can be cached, and should report if the path was finite?

[reed1, 2013-04-26 19:49:52]

lgtm

[caryclark, 2013-04-26 19:51:23]

Committed patchset #2 manually as r8882 (presubmit successful).