Heartbeat implementation of dart:mirrors.

runtime/tests/vm/src/IsolateMirrorBusyTest.dart

Index: runtime/tests/vm/src/IsolateMirrorBusyTest.dart
===================================================================
--- runtime/tests/vm/src/IsolateMirrorBusyTest.dart	(revision 0)
+++ runtime/tests/vm/src/IsolateMirrorBusyTest.dart	(revision 0)
@@ -0,0 +1,32 @@
+// 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.
+//
+// Dart test program for checking implemention of IsolateMirror.
+
+#library('IsolateMirrorBusyTest');
+
+#import('dart:mirrors');

[siva, 2012/02/18 01:25:55]

Since mirrors are part of the corelib at least in the current implementation is
this import needed. I am not sure if it is a good thing that this works.
It gets really confusing if somebody uses a prefix here.
e.g: #import('dart:mirrors', prefix:'core_mirror');

[turnidge, 2012/03/07 20:00:14]

Has your opinion on this changed after the recent dart:isolate cl?

[siva, 2012/03/08 22:24:02]

We are treating 'dart:mirrors' as an external library now and not part of
corelib so this import is ok.

+
+class BusyIsolate extends Isolate {
+  void busy() {
+    // TODO(turnidge): Get rid of this function once we check for
+    // interrupts on backwards branches.
+  }
+  void main() {
+    while (true) {
+      busy();
+    }
+  }
+}
+
+void testIsolateMirror(port) {
+  isolateMirrorOf(port).then((IsolateMirror mirror) {
+      Expect.isTrue(mirror.debugName.contains("BusyIsolate"));
+    });
+}
+
+void main() {
+  // Test that I can reflect on a busy isolate.
+  new BusyIsolate().spawn().then(testIsolateMirror);
+}