Remove deprecated ms arguments for Timer and Future.

runtime/lib/timer_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.
 
-typedef void _TimerCallback0();
-typedef void _TimerCallback1(Timer timer);
-
 patch class Timer {
-  /* patch */ factory Timer(var duration, Function callback) {
-    // TODO(floitsch): remove these checks when we remove the deprecated
-    // millisecond argument and the 1-argument callback. Also remove
-    // the int-test below.
-    if (callback is! _TimerCallback0 && callback is! _TimerCallback1) {
-      throw new ArgumentError(callback);
-    }
-    int milliseconds = duration is int ? duration : duration.inMilliseconds;
-    if (milliseconds < 0) milliseconds = 0;
-    _TimerCallback1 oneArgumentCallback =
-        callback is _TimerCallback1 ? callback : (_) { callback(); };
+  /* patch */ factory Timer(Duration duration, void callback()) {
     if (_TimerFactory._factory == null) {
       throw new UnsupportedError("Timer interface not supported.");
     }
-    return _TimerFactory._factory(milliseconds, oneArgumentCallback, false);
+    int milliseconds = duration.inMilliseconds;
+    if (milliseconds < 0) milliseconds = 0;

[Lasse Reichstein Nielsen, 2013/02/28 11:14:48]

Is that possible? I would have thought Duration disallowed it.
Or is Duration really a TimeDelta, so it can have a direction too?

[floitsch, 2013/02/28 13:24:05]

Duration is a time-delta. We wouldn't even be able to enforce it, since it's a
const-class.

+    return _TimerFactory._factory(milliseconds, (_) { callback(); }, false);
   }
 
-  /**
-   * Creates a new repeating timer. The [callback] is invoked every
-   * [milliseconds] millisecond until cancelled.
-   */
-  /* patch */ factory Timer.repeating(var duration,
+  /* patch */ factory Timer.repeating(Duration duration,
                                       void callback(Timer timer)) {
     if (_TimerFactory._factory == null) {
       throw new UnsupportedError("Timer interface not supported.");
     }
-    // TODO(floitsch): remove this check when we remove the deprecated
-    // millisecond argument.
-    int milliseconds = duration is int ? duration : duration.inMilliseconds;
+    int milliseconds = duration.inMilliseconds;
     if (milliseconds < 0) milliseconds = 0;
     return _TimerFactory._factory(milliseconds, callback, true);
   }
 }
 
 typedef Timer _TimerFactoryClosure(int milliseconds,
                                    void callback(Timer timer),
                                    bool repeating);
 
 class _TimerFactory {
   static _TimerFactoryClosure _factory;
 }
 
 // TODO(ahe): Warning: this is NOT called by Dartium. Instead, it sets
 // [_TimerFactory._factory] directly.
 void _setTimerFactoryClosure(_TimerFactoryClosure closure) {
   _TimerFactory._factory = closure;
 }