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

Unified Diff: tests/isolate/function_send_test.dart

Issue 742873002: Isolates: allow sending of arbitrary objects in dart2js. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Support CSP mode. Created 6 years 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
« no previous file with comments | « sdk/lib/_internal/compiler/js_lib/shared/embedded_names.dart ('k') | tests/isolate/isolate.status » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/isolate/function_send_test.dart
diff --git a/tests/isolate/function_send_test.dart b/tests/isolate/function_send_test.dart
index 95f4f93171c7156e03a8a27da80eae206da08bff..51d26ba58ebab56be3757ad1274cef2dc82a5aaa 100644
--- a/tests/isolate/function_send_test.dart
+++ b/tests/isolate/function_send_test.dart
@@ -153,13 +153,12 @@ void testUnsendable(name, func) {
asyncStart();
var noReply = new RawReceivePort((_) { throw "Unexpected message: $_"; });
- // Currently succeedes incorrectly in dart2js.
- Expect.throws(() { /// 01: ok
- noReply.sendPort.send(func); /// 01: continued
- }, null, "send direct"); /// 01: continued
- Expect.throws(() { /// 01: continued
- noReply.sendPort.send([func]); /// 01: continued
- }, null, "send wrapped"); /// 01: continued
+ Expect.throws(() {
+ noReply.sendPort.send(func);
+ }, null, "send direct");
+ Expect.throws(() {
+ noReply.sendPort.send([func]);
+ }, null, "send wrapped");
scheduleMicrotask(() {
noReply.close();
asyncEnd();
« no previous file with comments | « sdk/lib/_internal/compiler/js_lib/shared/embedded_names.dart ('k') | tests/isolate/isolate.status » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698