Use isfinite() on all POSIX platforms

Use isfinite() on all POSIX platforms

Based on some research online, it appears this should work everywhere.

BUG=None
TEST=None


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=146361

[stuartmorgan, 2012-07-12 12:33:23]

This is the follow-up to our discussion in
http://codereview.chromium.org/10704126/

I did some more looking, and found
https://bugs.launchpad.net/inkscape/+bug/167782
that claims that math.h on Solaris will include isfinite() if
__C99FEATURES__
is set, which it is for Solaris in common.gypi

So with a bit of luck this will work everywhere.

[Mark Mentovai, 2012-07-12 13:12:26]

LGTM

[commit-bot: I haz the power, 2012-07-12 13:14:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/stuartmorgan@chromium.org/10700183/1

[commit-bot: I haz the power, 2012-07-12 13:26:20]

Try job failure for 10700183-1 (retry) on android for steps "compile, build"
(clobber build).
It's a second try, previously, steps "compile, build" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android&nu...

[stuartmorgan, 2012-07-12 13:34:23]

Huh. Checking with some local Android folks...

[stuartmorgan, 2012-07-12 14:31:07]

Apparently Android has a Solaris-style bug where an include chain brings in a
cmath.h that does wacky undef-ing.

I've restored the old code path just for Android, with a bug reference. This may
mean Solaris will break too; if they do they can opt in to the Android path but
I'll hope for the best to keep the workaround as minimal as we know we need.

[Mark Mentovai, 2012-07-12 14:53:13]

LGTM again

[commit-bot: I haz the power, 2012-07-12 14:53:35]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/stuartmorgan@chromium.org/10700183/8002

[commit-bot: I haz the power, 2012-07-12 16:25:50]

Change committed as 146361