Fix type errors in math.min() and math.max().

pkg/dev_compiler/tool/input_sdk/patch/math_patch.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 // Patch file for dart:math library.
 import 'dart:_foreign_helper' show JS;
 import 'dart:_js_helper' show patch, checkNum;
 import 'dart:typed_data' show ByteData;
 
 @patch
+num/*=T*/ min/*<T extends num>*/(num/*=T*/ a, num/*=T*/ b)
+  => JS('num', r'Math.min(#, #)', checkNum(a), checkNum(b)) as num/*=T*/;

[vsm, 2016/09/30 19:36:33]

Do we actually need the checkNum?  Would a null check be sufficient?

[Bob Nystrom, 2016/09/30 19:43:14]

Good question. I was just following the nearby code (sqrt() et al.) that also do
checkNum(). Should I change it? What about the others?

+
+@patch
+num/*=T*/ max/*<T extends num>*/(num/*=T*/ a, num/*=T*/ b)
+  => JS('num', r'Math.max(#, #)', checkNum(a), checkNum(b)) as num/*=T*/;
+
+@patch
 double sqrt(num x)
   => JS('num', r'Math.sqrt(#)', checkNum(x));
 
 @patch
 double sin(num x)
   => JS('num', r'Math.sin(#)', checkNum(x));
 
 @patch
 double cos(num x)
   => JS('num', r'Math.cos(#)', checkNum(x));
@@ skipping 300 matching lines @@
       // last range of k*max .. 256**byteCount.
       // TODO: Consider picking a higher byte count if the last range is a
       // significant portion of the entire range - a 50% chance of having
       // to use two more bytes is no worse than always using one more.
       if (random - result + max < randomLimit) {
         return result;
       }
     }
   }
 }