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

Side by Side Diff: tests/lib_strong/async/zone_register_callback_test.dart

Issue 2963743002: Revert "Make Zone API strong mode clean." (Closed)
Patch Set: Created 3 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2013, 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 import 'package:expect/expect.dart'; 5 import 'package:expect/expect.dart';
6 import 'package:async_helper/async_helper.dart'; 6 import 'package:async_helper/async_helper.dart';
7 import 'dart:async'; 7 import 'dart:async';
8 8
9 main() { 9 main() {
10 Completer done = new Completer(); 10 Completer done = new Completer();
11 11
12 var valueToCapture; 12 var valueToCapture;
13 var restoredValue; 13 var restoredValue;
14 14
15 Expect.identical(Zone.ROOT, Zone.current); 15 Expect.identical(Zone.ROOT, Zone.current);
16 Zone forked = Zone.current.fork(specification: new ZoneSpecification( 16 Zone forked = Zone.current.fork(specification: new ZoneSpecification(
17 registerCallback: 17 registerCallback: (Zone self, ZoneDelegate parent, Zone origin, f()) {
18 <R>(Zone self, ZoneDelegate parent, Zone origin, R f()) {
19 // The zone is still the same as when origin.run was invoked, which 18 // The zone is still the same as when origin.run was invoked, which
20 // is the root zone. (The origin zone hasn't been set yet). 19 // is the root zone. (The origin zone hasn't been set yet).
21 Expect.identical(Zone.current, Zone.ROOT); 20 Expect.identical(Zone.current, Zone.ROOT);
22 // Note that not forwarding is completely legal, though not encouraged. 21 // Note that not forwarding is completely legal, though not encouraged.
23 var capturedValue = valueToCapture; 22 var capturedValue = valueToCapture;
24 return parent.registerCallback(origin, () { 23 return parent.registerCallback(origin, () {
25 restoredValue = capturedValue; 24 restoredValue = capturedValue;
26 return f(); 25 return f();
27 }); 26 });
28 })); 27 }));
29 28
30 valueToCapture = 499; 29 valueToCapture = 499;
31 var fun = () => 99; 30 var fun = () => 99;
32 var registered = forked.registerCallback(fun); 31 var registered = forked.registerCallback(fun);
33 Expect.isFalse(identical(fun, registered)); 32 Expect.isFalse(identical(fun, registered));
34 33
35 // It is legal to invoke the callback in a different zone. This is, of course, 34 // It is legal to invoke the callback in a different zone. This is, of course,
36 // extremely discouraged. 35 // extremely discouraged.
37 var result = registered(); 36 var result = registered();
38 Expect.equals(99, result); 37 Expect.equals(99, result);
39 Expect.equals(499, restoredValue); 38 Expect.equals(499, restoredValue);
40 } 39 }
OLDNEW
« no previous file with comments | « tests/lib_strong/async/zone_fork_test.dart ('k') | tests/lib_strong/async/zone_register_callback_unary_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698