Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(428)

Unified Diff: runtime/vm/isolate_test.cc

Issue 10837070: Remove old isolate API and update all code in the repository to use (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Fix namespace comment. Created 8 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: runtime/vm/isolate_test.cc
diff --git a/runtime/vm/isolate_test.cc b/runtime/vm/isolate_test.cc
index 7655fad7c070837e9ceeeaff396d2d6d5cb4d9ab..08c1762dec1134c7cc61187beb9269be314e3864 100644
--- a/runtime/vm/isolate_test.cc
+++ b/runtime/vm/isolate_test.cc
@@ -17,35 +17,4 @@ UNIT_TEST_CASE(IsolateCurrent) {
delete isolate;
}
-
-// Only ia32 and x64 can run dart execution tests.
-#if defined(TARGET_ARCH_IA32) || defined(TARGET_ARCH_X64)
-// Unit test case to verify error during isolate spawning (application classes
-// not loaded into the isolate).
-TEST_CASE(IsolateSpawn) {
Mads Ager (google) 2012/08/02 15:04:20 Looking at this test more with the help of Ivan it
Mads Ager (google) 2012/08/03 05:51:35 Done. Reintroduced. Added the test for a NULL call
- const char* kScriptChars =
- "#import('dart:isolate');\n"
- "class SpawnNewIsolate extends Isolate {\n"
- " SpawnNewIsolate() : super() { }\n"
- " void main() {\n"
- " }\n"
- "}\n"
- "int testMain() {\n"
- " try {\n"
- " new SpawnNewIsolate().spawn().then(function(SendPort port) {\n"
- " });\n"
- " } catch (var e) {\n"
- " throw;\n"
- " }\n"
- " return 0;\n"
- "}\n";
- Dart_Handle lib = TestCase::LoadTestScript(kScriptChars, NULL);
- Dart_Handle result = Dart_Invoke(lib, Dart_NewString("testMain"), 0, NULL);
- EXPECT(Dart_IsError(result));
- EXPECT(Dart_ErrorHasException(result));
- Dart_Handle exception_result = Dart_ErrorGetException(result);
- EXPECT_VALID(exception_result);
-}
-#endif // defined(TARGET_ARCH_IA32) || defined(TARGET_ARCH_X64).
-
} // namespace dart

Powered by Google App Engine
This is Rietveld 408576698