Add isActive field on Timer.

tools/dom/src/native_DOMImplementation.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.
 
 part of html;
 
 class _Utils {
   static double dateTimeToDouble(DateTime dateTime) =>
       dateTime.millisecondsSinceEpoch.toDouble();
   static DateTime doubleToDateTime(double dateTime) {
@@ skipping 180 matching lines @@
   });
 }
 
 final _printClosure = window.console.log;
 final _pureIsolatePrintClosure = (s) {
   _sendToHelperIsolate([_PRINT, s], null);
 };
 
 final _forwardingPrintClosure = _Utils.forwardingPrint;
 
-class _Timer implements Timer {
-  final canceller;
+ class _Timer implements Timer {

[floitsch, 2013/07/02 11:27:19]

Remove leading space.

[zarah, 2013/07/02 11:43:12]

Done.

+  final _canceler;

[floitsch, 2013/07/02 11:27:19]

Can not be final, since you change it.

[zarah, 2013/07/02 11:43:12]

Done, of course!

 
-  _Timer(this.canceller);
+  _Timer(int milliSeconds, void callback(Timer timer), bool repeating) {
 
-  void cancel() { canceller(); }
+    if (repeating) {
+      int id = window._setInterval(() {
+        _canceler = null;

[floitsch, 2013/07/02 11:27:19]

No. Don't remove the canceler for repeating timers.

[zarah, 2013/07/02 11:43:12]

Done.

+        callback(this);
+      }, milliSeconds);)
+      _canceler = () => window._clearInterval(id);
+    } else {
+      int id = window._setTimeout(() {
+        _canceler = null;
+        callback(this);
+      }, milliSeconds); )
+      _canceler = window._clearTimeout(id);

[floitsch, 2013/07/02 11:27:19]

This must be a closure.

[zarah, 2013/07/02 11:43:12]

Done.

+    }
+  }
+
+  void cancel() {
+    if (_canceler != null) {
+      _canceler();
+    }
+    _canceler = null;
+  }
+
+  bool get isActive => _canceler != null;
 }
 
-get _timerFactoryClosure => (int milliSeconds, void callback(Timer timer), bool repeating) {
-  var maker;
-  var canceller;
-  if (repeating) {
-    maker = window._setInterval;
-    canceller = window._clearInterval;
-  } else {
-    maker = window._setTimeout;
-    canceller = window._clearTimeout;
-  }
-  Timer timer;
-  final int id = maker(() { callback(timer); }, milliSeconds);
-  timer = new _Timer(() { canceller(id); });
-  return timer;
+get _timerFactoryClosure =>
+    (int milliSeconds, void callback(Timer timer), bool repeating) {
+  return new _Timer(milliseconds, callback, repeating);
 };
 
+
 class _PureIsolateTimer implements Timer {
+  bool _isDone = false;

[floitsch, 2013/07/02 11:27:19]

Rename _isDone to _isActive

[zarah, 2013/07/02 11:43:12]

Done.

   final ReceivePort _port = new ReceivePort();
   SendPort _sendPort; // Effectively final.
 
   static SendPort _SEND_PORT;
 
   _PureIsolateTimer(int milliSeconds, callback, repeating) {
     _sendPort = _port.toSendPort();
     _port.receive((msg, replyTo) {
       assert(msg == _TIMER_PING);
+      _isDone = !repeating;
       callback(this);
       if (!repeating) _cancel();
     });
 
     _send([_NEW_TIMER, milliSeconds, repeating]);
   }
 
   void cancel() {
     _cancel();
     _send([_CANCEL_TIMER]);
   }
 
   void _cancel() {
+    _isDone = true;
     _port.close();
   }
 
   _send(msg) {
     _sendToHelperIsolate(msg, _sendPort);
   }
+
+  bool get isActive => !_isDone;
 }
 
 get _pureIsolateTimerFactoryClosure =>
     ((int milliSeconds, void callback(Timer time), bool repeating) =>
         new _PureIsolateTimer(milliSeconds, callback, repeating));