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

Unified 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: removed dead files 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 side-by-side diff with in-line comments
Download patch
« frog/member.dart ('K') | « frog/minfrog ('k') | frog/value.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: frog/type.dart
diff --git a/frog/type.dart b/frog/type.dart
index 9f612b73c7dafab2ff0ae27db60dde90fafa48b0..677c8b29456bb90b0d5e6c7e8861fd3b3602b3a8 100644
--- a/frog/type.dart
+++ b/frog/type.dart
@@ -107,8 +107,8 @@ class Type extends Element {
Map<String, Member> getAllMembers() => {};
- /** The native name of this element if it has one. */
- String get nativeName() => isNative ? definition.nativeType.name : null;
+ /** The native name of this element if it has one, otherwise the JS name. */
+ String get nativeName() => isNative ? definition.nativeType.name : jsname;
/**
* Avoid the native name if hidden native. It might exist on some browsers and
« frog/member.dart ('K') | « frog/minfrog ('k') | frog/value.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698