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

Unified Diff: runtime/lib/mirrors_impl.dart

Issue 19512003: Implement VariableMirror creation with non-API code. To break circular initialization, have them fi… (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 7 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 side-by-side diff with in-line comments
Download patch
Index: runtime/lib/mirrors_impl.dart
===================================================================
--- runtime/lib/mirrors_impl.dart (revision 25253)
+++ runtime/lib/mirrors_impl.dart (working copy)
@@ -1037,6 +1037,9 @@
var _type;
TypeMirror get type {
+ if (_type == null) {
+ _type = _VariableMirror_type(_reflectee);
+ }
if (_type is! Mirror) {
_type = _type.resolve(mirrors);
}
@@ -1047,6 +1050,9 @@
final bool isFinal;
String toString() => "VariableMirror on '${_n(simpleName)}'";
+
+ static _VariableMirror_type(reflectee)
+ native "VariableMirror_type";
}
class _LocalParameterMirrorImpl extends _LocalVariableMirrorImpl
@@ -1096,6 +1102,13 @@
}
String toString() => "TypeMirror on '${_n(simpleName)}'";
+
+ operator ==(other) {
rmacnak 2013/07/20 00:46:52 Allow tests to pass which previously relied on the
siva 2013/07/22 05:25:32 Please add an issue to track this temporary workar
rmacnak 2013/07/22 19:05:09 Done.
+ if (other is! _SpecialTypeMirrorImpl) {
+ return false;
+ }
+ return this.simpleName == other.simpleName;
+ }
}
class _Mirrors {

Powered by Google App Engine
This is Rietveld 408576698