If an unhandled exception occurs in Dart_RunLoop, pass it out to the caller.

runtime/vm/dart_api_impl_test.cc

 // Copyright (c) 2011, 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.
 
 #include "include/dart_api.h"
 
 #include "vm/assert.h"
 #include "vm/dart_api_impl.h"
 #include "vm/dart_api_state.h"
 #include "vm/unit_test.h"
@@ skipping 2748 matching lines @@
 
 
 static bool RunLoopTestCallback(void* data, char** error) {
   const char* kScriptChars =
       "#import('builtin');\n"
       "class MyIsolate extends Isolate {\n"
       "  MyIsolate() : super() { }\n"
       "  void main() {\n"
       "    port.receive((message, replyTo) {\n"
       "      if (message) {\n"
-      "        throw new Exception('MakeVMExit');\n"
+      "        throw new Exception('MakeChildExit');\n"
       "      } else {\n"
       "        replyTo.call('hello');\n"
       "        port.close();\n"
       "      }\n"
       "    });\n"
       "  }\n"
       "}\n"
       "\n"
-      "void main(message) {\n"
+      "void main(exc_child, exc_parent) {\n"
       "  new MyIsolate().spawn().then((port) {\n"
-      "    port.call(message).receive((message, replyTo) {\n"
+      "    port.call(exc_child).receive((message, replyTo) {\n"
       "      if (message != 'hello') throw new Exception('ShouldNotHappen');\n"
+      "      if (exc_parent) throw new Exception('MakeParentExit');\n"
       "    });\n"
       "  });\n"
       "}\n";
 
   if (Dart_CurrentIsolate() != NULL) {
     Dart_ExitIsolate();
   }
   Dart_Isolate isolate = TestCase::CreateTestIsolate();
   ASSERT(isolate != NULL);
   Dart_EnterScope();
   Dart_Handle url = Dart_NewString(TestCase::url());
   Dart_Handle source = Dart_NewString(kScriptChars);
   Dart_Handle lib = Dart_LoadScript(url, source, TestCase::library_handler);
   EXPECT_VALID(lib);
   Dart_ExitScope();
   return true;
 }
 
 
 // Common code for RunLoop_Success/RunLoop_Failure.
-static void RunLoopTest(bool throw_exception) {
+static void RunLoopTest(bool throw_exception_child,
+                        bool throw_exception_parent) {
   Dart_IsolateCreateCallback saved = Isolate::CreateCallback();
   Isolate::SetCreateCallback(RunLoopTestCallback);
   RunLoopTestCallback(NULL, NULL);
 
   Dart_EnterScope();
   Dart_Handle lib = Dart_LookupLibrary(Dart_NewString(TestCase::url()));
   EXPECT_VALID(lib);
 
   Dart_Handle result;
-  Dart_Handle args[1];
-  args[0] = (throw_exception ? Dart_True() : Dart_False());
+  Dart_Handle args[2];
+  args[0] = (throw_exception_child ? Dart_True() : Dart_False());
+  args[1] = (throw_exception_parent ? Dart_True() : Dart_False());
   result = Dart_InvokeStatic(lib,
                              Dart_NewString(""),
                              Dart_NewString("main"),
-                             1,
+                             2,
                              args);
   EXPECT_VALID(result);
   result = Dart_RunLoop();
-  EXPECT_VALID(result);
+  if (throw_exception_parent) {
+    EXPECT(Dart_IsError(result));
+    // TODO(turnidge): Once EXPECT_SUBSTRING is submitted use it here.
+  } else {
+    EXPECT_VALID(result);
+  }
 
   Dart_ExitScope();
   Dart_ShutdownIsolate();
 
   Isolate::SetCreateCallback(saved);
 }
 
 
 UNIT_TEST_CASE(RunLoop_Success) {
-  RunLoopTest(false);
+  RunLoopTest(false, false);
 }
 
 
-UNIT_TEST_CASE(RunLoop_Exception) {
-  RunLoopTest(true);
+// This test exits the vm.  Listed as FAIL in vm.status.
+UNIT_TEST_CASE(RunLoop_ExceptionChild) {
+  RunLoopTest(true, false);
+}
+
+
+UNIT_TEST_CASE(RunLoop_ExceptionParent) {
+  RunLoopTest(false, true);
 }
 
 #endif  // TARGET_ARCH_IA32.
 
 }  // namespace dart