Enable isolate tests with leg and legium.

lib/isolate/frog/isolateimpl.dart

Index: lib/isolate/frog/isolateimpl.dart
===================================================================
--- lib/isolate/frog/isolateimpl.dart	(revision 5352)
+++ lib/isolate/frog/isolateimpl.dart	(working copy)
@@ -197,6 +197,16 @@
   }
 }
 
+// We don't want to import the DOM library just because of window.setTimeout,
+// so we reconstruct the Window class here. The only conflict that could happen
+// with the other DOMWindow class would be because of subclasses.
+// Currently, none of the two Dart classes have subclasses.

[ngeoffray, 2012/03/12 23:18:36]

This is more of a cleanup than a hack IMO. Siggi, Eli let me know what you
think.

[Siggi Cherem (dart-lang), 2012/03/12 23:40:58]

looks good to me. I'm working on adding Timer to corelib, which is basically
based on this. Hopefully then we can hide this code behind the Timer interface. 

I don't plan to implement Timer outside the browser, so we will still need to
detect if it is available, for instance checking if new Timer(x, callback)
throws an exception.

[ngeoffray, 2012/03/13 12:51:37]

Sounds good.
I don't understand: if you're adding something to the corelib, it ought to be
available to both browser and standalone. Or are you saying that the actual
implementation of Timer in standalone will be implemented by someone else?

[Siggi Cherem (dart-lang), 2012/03/14 23:23:12]

It would be supported standalone in the VM, but not when compiling with frog to
run in d8. In nodejs we had setTimeout available, but I haven't found something
equivalent for d8. I was leaning to say that in such case Timer is not
available.

[ngeoffray, 2012/03/15 09:30:56]

Oh I see. So I guess you want to throw a runtime exception by doing something
like:

class Timer {
   Timer() {
      if (JS_CODE("typeof window == 'undefined')) {
        throw 'Timer not supported';
      }
      ....
   }
}

+typedef void _TimeoutHandler();
+class _Window native "@*DOMWindow" {
+  int setTimeout(_TimeoutHandler handler, int timeout) native;

[Siggi Cherem (dart-lang), 2012/03/12 23:40:58]

is this _TimeoutHandler needed, or can we simply do:
int setTimeout(void handler(), int timeout) native;
?

[ngeoffray, 2012/03/13 12:51:37]

Leg does not handle function type parameters very well right now. We're fixing
it.

[Siggi Cherem (dart-lang), 2012/03/14 23:23:12]

good to know. Thx.

+}
+_Window get _window() native
+    """return typeof window != 'undefined' ? window : (void 0);""";
 
 /** Represent the event loop on a javascript thread (DOM or worker). */
 class _EventLoop {
@@ -224,23 +234,16 @@
     return true;
   }
 
-  /** Function equivalent to [:window.setTimeout:] when available, or null. */
-  static Function _wrapSetTimeout() native """
-      return typeof window != 'undefined' ?
-          function(a, b) { window.setTimeout(a, b); } : undefined;
-  """;
-
   /**
    * Runs multiple iterations of the run-loop. If possible, each iteration is
    * run asynchronously.
    */
   void _runHelper() {
-    final setTimeout = _wrapSetTimeout();
-    if (setTimeout != null) {
+    if (_window != null) {
       // Run each iteration from the browser's top event loop.
       void next() {
         if (!runIteration()) return;
-        setTimeout(next, 0);
+        _window.setTimeout(next, 0);
       }
       next();
     } else {