Reinstate type checks in Math.min and Math.max

Reinstate type checks in Math.min and Math.max

See Issue 11906 for discussion.

R=srdjan@google.com

Committed: https://code.google.com/p/dart/source/detail?r=27980

[sra1, 2013-09-14 00:47:09]

Srdjan - is there anything else required for the VM?

[sra1, 2013-09-17 21:26:58]

Ping

[sra1, 2013-09-19 22:40:43]

Siva - Srdjan is out, can you review?
Specifically, is changing the fingerprint all I need to do?

[siva, 2013-09-20 00:36:16]

I somehow don't like these additional redundant type tests that will be flagged
in checked mode anyways.

Can we wait till Srdjan is back on this for his opinion. He should be back on
Monday.

[sra1, 2013-09-20 01:13:23]

On 2013/09/20 00:36:16, siva wrote:
> I somehow don't like these additional redundant type tests that will be
flagged
> in checked mode anyways.
> 
> Can we wait till Srdjan is back on this for his opinion. He should be back on
> Monday.

See Issue 11906 for the discussion, where Lars agrees to do this.

[srdjan, 2013-09-26 22:09:50]

lgtm

https://chromiumcodereview.appspot.com/23658055/diff/7001/sdk/lib/math/math.dart
File sdk/lib/math/math.dart (right):

https://chromiumcodereview.appspot.com/23658055/diff/7001/sdk/lib/math/math.d...
sdk/lib/math/math.dart:68: if (b is! num) throw new ArgumentError(b);
Please add test that verifies this new/old behavior.

[sra1, 2013-09-27 00:45:13]

Committed patchset #3 manually as r27980 (presubmit successful).