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

Side by Side Diff: frog/type.dart

Issue 9110027: Some cleanups to Frog to avoid looking up its builtin types too much (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: updated Created 8 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
OLDNEW
1 // Copyright (c) 2011, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2011, 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 class Type extends Element { 5 class Type extends Element {
6 bool isTested = false; 6 bool isTested = false;
7 bool isChecked = false; 7 bool isChecked = false;
8 bool isWritten = false; 8 bool isWritten = false;
9 9
10 /** 10 /**
(...skipping 89 matching lines...) Expand 10 before | Expand all | Expand 10 after
100 // the absence of type parameters. 100 // the absence of type parameters.
101 Collection<Type> get typeArgsInOrder() => null; 101 Collection<Type> get typeArgsInOrder() => null;
102 DefinedType get genericType() => this; 102 DefinedType get genericType() => this;
103 103
104 // TODO(jmesserly): what should these do for ParameterType? 104 // TODO(jmesserly): what should these do for ParameterType?
105 List<Type> get interfaces() => null; 105 List<Type> get interfaces() => null;
106 Type get parent() => null; 106 Type get parent() => null;
107 107
108 Map<String, Member> getAllMembers() => {}; 108 Map<String, Member> getAllMembers() => {};
109 109
110 /** The native name of this element if it has one. */ 110 /** The native name of this element if it has one, otherwise the JS name. */
111 String get nativeName() => isNative ? definition.nativeType.name : null; 111 String get nativeName() => isNative ? definition.nativeType.name : jsname;
112 112
113 /** 113 /**
114 * Avoid the native name if hidden native. It might exist on some browsers and 114 * Avoid the native name if hidden native. It might exist on some browsers and
115 * we want to use it if it does. 115 * we want to use it if it does.
116 */ 116 */
117 bool get avoidNativeName() => isHiddenNativeType; 117 bool get avoidNativeName() => isHiddenNativeType;
118 118
119 bool _hasNativeSubtypes; 119 bool _hasNativeSubtypes;
120 bool get hasNativeSubtypes() { 120 bool get hasNativeSubtypes() {
121 if (_hasNativeSubtypes == null) { 121 if (_hasNativeSubtypes == null) {
(...skipping 1223 matching lines...) Expand 10 before | Expand all | Expand 10 after
1345 } else { 1345 } else {
1346 break; 1346 break;
1347 } 1347 }
1348 } 1348 }
1349 if (name.startsWith('=')) { 1349 if (name.startsWith('=')) {
1350 name = name.substring(1); 1350 name = name.substring(1);
1351 isSingleton = true; 1351 isSingleton = true;
1352 } 1352 }
1353 } 1353 }
1354 } 1354 }
OLDNEW
« no previous file with comments | « frog/minfrog ('k') | frog/value.dart » ('j') | utils/apidoc/html_diff.dart » ('J')

Powered by Google App Engine
This is Rietveld 408576698