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

Side by Side Diff: tests/isolate/illegal_msg_mirror_test.dart

Issue 51393002: Raise an exception instead of crashing when attempting to pass a MirrorReference between isolates. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 1 month 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « tests/isolate/illegal_msg_function_test.dart ('k') | tests/isolate/illegal_msg_test.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 library illegal_msg_tests; 5 library illegal_msg_mirror_test;
6
6 import "package:expect/expect.dart"; 7 import "package:expect/expect.dart";
7 import 'dart:isolate'; 8 import "dart:isolate";
9 import "dart:async" show Future;
8 import "package:async_helper/async_helper.dart"; 10 import "package:async_helper/async_helper.dart";
11 import "dart:mirrors";
9 12
10 funcFoo(x) => x + 2; 13 class Class {
14 method() {}
15 }
11 16
12 echo(sendPort) { 17 echo(sendPort) {
13 var port = new ReceivePort(); 18 var port = new ReceivePort();
14 sendPort.send(port.sendPort); 19 sendPort.send(port.sendPort);
15 port.listen((msg) { 20 port.listen((msg) {
16 sendPort.send("echoing ${msg(1)}}"); 21 sendPort.send("echoing ${msg(1)}}");
17 }); 22 });
18 } 23 }
19 24
20 main() { 25 main() {
26 var methodMirror = reflectClass(Class).declarations[#method];
27
21 ReceivePort port = new ReceivePort(); 28 ReceivePort port = new ReceivePort();
22 Future spawn = Isolate.spawn(echo, port.sendPort); 29 Future spawn = Isolate.spawn(echo, port.sendPort);
23 var caught_exception = false; 30 var caught_exception = false;
24 var stream = port.asBroadcastStream(); 31 var stream = port.asBroadcastStream();
25 asyncStart(); 32 asyncStart();
26 stream.first.then((snd) { 33 stream.first.then((snd) {
27 try { 34 try {
28 snd.send(funcFoo); 35 snd.send(methodMirror);
29 } catch (e) { 36 } catch (e) {
30 caught_exception = true; 37 caught_exception = true;
31 } 38 }
32 39
33 if (caught_exception) { 40 if (caught_exception) {
34 port.close(); 41 port.close();
35 } else { 42 } else {
36 asyncStart(); 43 asyncStart();
37 stream.first.then((msg) { 44 stream.first.then((msg) {
38 print("from worker ${msg}"); 45 print("from worker ${msg}");
39 asyncEnd(); 46 asyncEnd();
40 }); 47 });
41 } 48 }
42 Expect.isTrue(caught_exception); 49 Expect.isTrue(caught_exception);
43 asyncEnd(); 50 asyncEnd();
44 }); 51 });
45 } 52 }
OLDNEW
« no previous file with comments | « tests/isolate/illegal_msg_function_test.dart ('k') | tests/isolate/illegal_msg_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698