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

Side by Side Diff: pkg/serialization/lib/src/mirrors_helpers.dart

Issue 11801008: Fix tests for VM in checked mode. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Add missing expectAsync1 Created 7 years, 11 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 | Annotate | Revision Log
« no previous file with comments | « pkg/oauth2/test/client_test.dart ('k') | runtime/lib/integers_patch.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) 2012, 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 /** 5 /**
6 * Provides some additional convenience methods on top of the basic mirrors 6 * Provides some additional convenience methods on top of the basic mirrors
7 */ 7 */
8 library mirrors_helpers; 8 library mirrors_helpers;
9 9
10 // Import and re-export mirrors here to minimize both dependence on mirrors 10 // Import and re-export mirrors here to minimize both dependence on mirrors
11 // and the number of times we have to be told that mirrors aren't finished yet. 11 // and the number of times we have to be told that mirrors aren't finished yet.
12 import 'dart:mirrors'; 12 import 'dart:mirrors';
13 export 'dart:mirrors'; 13 export 'dart:mirrors';
14 import 'serialization_helpers.dart'; 14 import 'serialization_helpers.dart';
15 15
16 /** 16 /**
17 * Return a list of all the public fields of a class, including inherited 17 * Return a list of all the public fields of a class, including inherited
18 * fields. 18 * fields.
19 */ 19 */
20 List<VariableMirror> publicFields(ClassMirror mirror) { 20 List<VariableMirror> publicFields(ClassMirror mirror) {
21 var mine = mirror.variables.values.where( 21 var mine = mirror.variables.values.where(
22 (x) => !(x.isPrivate || x.isStatic)); 22 (x) => !(x.isPrivate || x.isStatic)).toList();
23 var mySuperclass = mirror.superclass; 23 var mySuperclass = mirror.superclass;
24 if (mySuperclass != mirror) { 24 if (mySuperclass != mirror) {
25 return append(publicFields(mirror.superclass), mine); 25 return append(publicFields(mirror.superclass), mine);
26 } else { 26 } else {
27 return mine; 27 return mine;
28 } 28 }
29 } 29 }
30 30
31 /** Return true if the class has a field named [name]. Note that this 31 /** Return true if the class has a field named [name]. Note that this
32 * includes private fields, but excludes statics. */ 32 * includes private fields, but excludes statics. */
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
70 return publicGetters(mirror).where((each) => 70 return publicGetters(mirror).where((each) =>
71 setters["${each.simpleName}="] != null).toList(); 71 setters["${each.simpleName}="] != null).toList();
72 } 72 }
73 73
74 /** 74 /**
75 * A particularly bad case of polyfill, because we cannot yet use type names 75 * A particularly bad case of polyfill, because we cannot yet use type names
76 * as literals, so we have to be passed an instance and then extract a 76 * as literals, so we have to be passed an instance and then extract a
77 * ClassMirror from that. Given a horrible name as an extra reminder to fix it. 77 * ClassMirror from that. Given a horrible name as an extra reminder to fix it.
78 */ 78 */
79 ClassMirror turnInstanceIntoSomethingWeCanUse(x) => reflect(x).type; 79 ClassMirror turnInstanceIntoSomethingWeCanUse(x) => reflect(x).type;
OLDNEW
« no previous file with comments | « pkg/oauth2/test/client_test.dart ('k') | runtime/lib/integers_patch.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698