Index: utils/apidoc/html_diff.dart |
diff --git a/utils/apidoc/html_diff.dart b/utils/apidoc/html_diff.dart |
index 415446c05066d5b03ff34c53aa20b7642b1a67a7..8d55542491cc7bed111a6d402cffa07eed7d1a00 100644 |
--- a/utils/apidoc/html_diff.dart |
+++ b/utils/apidoc/html_diff.dart |
@@ -10,10 +10,9 @@ |
#import('dart:coreimpl'); |
-#import('../../lib/dartdoc/frog/lang.dart'); |
-#import('../../lib/dartdoc/frog/file_system_vm.dart'); |
-#import('../../lib/dartdoc/frog/file_system.dart'); |
#import('../../lib/dartdoc/dartdoc.dart'); |
+#import('../../lib/dartdoc/mirrors/mirrors.dart'); |
+#import('../../lib/dartdoc/mirrors/mirrors_util.dart'); |
/** |
* A class for computing a many-to-many mapping between the types and |
@@ -44,45 +43,54 @@ |
class HtmlDiff { |
/** A map from `dart:dom_deprecated` members to corresponding |
* `dart:html` members. */ |
- final Map<Member, Set<Member>> domToHtml; |
+ final Map<MemberMirror, Set<MemberMirror>> domToHtml; |
/** A map from `dart:html` members to corresponding |
* `dart:dom_deprecated` members. */ |
- final Map<Member, Set<Member>> htmlToDom; |
+ final Map<MemberMirror, Set<MemberMirror>> htmlToDom; |
/** A map from `dart:dom_deprecated` types to corresponding |
- * `dart:html` types. */ |
- final Map<Type, Set<Type>> domTypesToHtml; |
+ * `dart:html` types. |
+ * TODO(johnniwinther): We use qualified names as keys, since mirrors |
+ * (currently) are not equal between different mirror systems. |
+ */ |
+ final Map<String, Set<InterfaceMirror>> domTypesToHtml; |
/** A map from `dart:html` types to corresponding |
- * `dart:dom_deprecated` types. */ |
- final Map<Type, Set<Type>> htmlTypesToDom; |
+ * `dart:dom_deprecated` types. |
+ * TODO(johnniwinther): We use qualified names as keys, since mirrors |
+ * (currently) are not equal between different mirror systems. |
+ */ |
+ final Map<String, Set<InterfaceMirror>> htmlTypesToDom; |
final CommentMap comments; |
/** If true, then print warning messages. */ |
final bool _printWarnings; |
- static Library dom; |
+ static Compilation _compilation; |
+ static MirrorSystem _mirrors; |
+ static LibraryMirror dom; |
/** |
* Perform static initialization of [world]. This should be run before |
* calling [HtmlDiff.run]. |
*/ |
- static void initialize() { |
- world.getOrAddLibrary('dart:dom_deprecated'); |
- world.getOrAddLibrary('dart:html'); |
- world.process(); |
+ static void initialize(String libDir) { |
+ _compilation = new Compilation.library( |
+ const <String>['dart:dom_deprecated', 'dart:html'], libDir); |
+ _mirrors = _compilation.mirrors(); |
- dom = world.libraries['dart:dom_deprecated']; |
+ // Find 'dart:dom_deprecated' by its library tag 'dom'. |
+ dom = findMirror(_mirrors.libraries(), 'dom'); |
} |
HtmlDiff([bool printWarnings = false]) : |
_printWarnings = printWarnings, |
- domToHtml = new Map<Member, Set<Member>>(), |
- htmlToDom = new Map<Member, Set<Member>>(), |
- domTypesToHtml = new Map<Type, Set<Type>>(), |
- htmlTypesToDom = new Map<Type, Set<Type>>(), |
+ domToHtml = new Map<MemberMirror, Set<MemberMirror>>(), |
+ htmlToDom = new Map<MemberMirror, Set<MemberMirror>>(), |
+ domTypesToHtml = new Map<String, Set<InterfaceMirror>>(), |
+ htmlTypesToDom = new Map<String, Set<InterfaceMirror>>(), |
comments = new CommentMap(); |
void warn(String s) { |
@@ -99,20 +107,23 @@ class HtmlDiff { |
* [HtmlDiff.initialize] should be called. |
*/ |
void run() { |
- final htmlLib = world.libraries['dart:html']; |
- for (Type htmlType in htmlLib.types.getValues()) { |
+ LibraryMirror htmlLib = findMirror(_mirrors.libraries(), 'html'); |
+ if (htmlLib === null) { |
+ warn('Could not find dart:html'); |
+ return; |
+ } |
+ for (InterfaceMirror htmlType in htmlLib.types().getValues()) { |
final domTypes = htmlToDomTypes(htmlType); |
if (domTypes.isEmpty()) continue; |
- htmlTypesToDom.putIfAbsent(htmlType, |
+ htmlTypesToDom.putIfAbsent(htmlType.qualifiedName(), |
() => new Set()).addAll(domTypes); |
domTypes.forEach((t) => |
- domTypesToHtml.putIfAbsent(t, () => new Set()).add(htmlType)); |
+ domTypesToHtml.putIfAbsent(t.qualifiedName(), |
+ () => new Set()).add(htmlType)); |
- final members = new List.from(htmlType.members.getValues()); |
- members.addAll(htmlType.constructors.getValues()); |
- htmlType.factories.forEach((f) => members.add(f)); |
- members.forEach((m) => _addMemberDiff(m, domTypes)); |
+ htmlType.declaredMembers().forEach( |
+ (_, m) => _addMemberDiff(m, domTypes)); |
} |
} |
@@ -122,16 +133,12 @@ class HtmlDiff { |
* `dart:dom_deprecated` [Type]s that correspond to [implMember]'s |
* defining [Type]. |
*/ |
- void _addMemberDiff(Member htmlMember, List<Type> domTypes) { |
- if (htmlMember.isProperty) { |
- if (htmlMember.canGet) _addMemberDiff(htmlMember.getter, domTypes); |
- if (htmlMember.canSet) _addMemberDiff(htmlMember.setter, domTypes); |
- } |
- |
+ void _addMemberDiff(MemberMirror htmlMember, List<TypeMirror> domTypes) { |
var domMembers = htmlToDomMembers(htmlMember, domTypes); |
if (htmlMember == null && !domMembers.isEmpty()) { |
- warn('dart:html member ${htmlMember.declaringType.name}.' |
- '${htmlMember.name} has no corresponding dart:html member.'); |
+ warn('dart:html member ' |
+ '${htmlMember.surroundingDeclaration().simpleName()}.' |
+ '${htmlMember.simpleName()} has no corresponding dart:html member.'); |
} |
if (htmlMember == null) return; |
@@ -145,20 +152,27 @@ class HtmlDiff { |
* [htmlType] from `dart:html`. This can be the empty list if no |
* correspondence is found. |
*/ |
- List<Type> htmlToDomTypes(Type htmlType) { |
- if (htmlType.name == null) return []; |
- final tags = _getTags(comments.find(htmlType.span)); |
- |
+ List<InterfaceMirror> htmlToDomTypes(InterfaceMirror htmlType) { |
+ if (htmlType.simpleName() == null) return []; |
+ final tags = _getTags(comments.find(htmlType.location())); |
if (tags.containsKey('domName')) { |
- var domNames = map(tags['domName'].split(','), (s) => s.trim()); |
+ var domNames = <String>[]; |
+ for (var s in tags['domName'].split(',')) { |
+ domNames.add(s.trim()); |
+ } |
if (domNames.length == 1 && domNames[0] == 'none') return []; |
- return map(domNames, (domName) { |
- final domType = dom.types[domName]; |
- if (domType == null) warn('no dart:dom_deprecated type named $domName'); |
- return domType; |
- }); |
+ var domTypes = <InterfaceMirror>[]; |
+ for (var domName in domNames) { |
+ final domType = findMirror(dom.types(), domName); |
+ if (domType == null) { |
+ warn('no dart:dom_deprecated type named $domName'); |
+ } else { |
+ domTypes.add(domType); |
+ } |
+ } |
+ return domTypes; |
} |
- return <Type>[]; |
+ return <InterfaceMirror>[]; |
} |
/** |
@@ -168,11 +182,15 @@ class HtmlDiff { |
* `dart:dom_deprecated` [Type]s that correspond to [implMember]'s |
* defining [Type]. |
*/ |
- Set<Member> htmlToDomMembers(Member htmlMember, List<Type> domTypes) { |
+ Set<MemberMirror> htmlToDomMembers(MemberMirror htmlMember, |
+ List<InterfaceMirror> domTypes) { |
if (htmlMember.isPrivate) return new Set(); |
- final tags = _getTags(comments.find(htmlMember.span)); |
+ final tags = _getTags(comments.find(htmlMember.location())); |
if (tags.containsKey('domName')) { |
- final domNames = map(tags['domName'].split(','), (s) => s.trim()); |
+ var domNames = <String>[]; |
+ for (var s in tags['domName'].split(',')) { |
+ domNames.add(s.trim()); |
+ } |
if (domNames.length == 1 && domNames[0] == 'none') return new Set(); |
final members = new Set(); |
domNames.forEach((name) { |
@@ -181,8 +199,11 @@ class HtmlDiff { |
if (name.contains('.')) { |
warn('no member $name'); |
} else { |
- final options = Strings.join( |
- map(domTypes, (t) => "${t.name}.$name"), ' or '); |
+ final options = <String>[]; |
+ for (var t in domTypes) { |
+ options.add('${t.simpleName()}.${name}'); |
+ } |
+ Strings.join(options, ' or '); |
warn('no member $options'); |
} |
} |
@@ -202,15 +223,19 @@ class HtmlDiff { |
* name (e.g. `Document.createElement`), in which case it's looked |
* up in `dart:dom_deprecated` and [defaultTypes] is ignored. |
*/ |
- Set<Member> _membersFromName(String name, List<Type> defaultTypes) { |
+ Set<MemberMirror> _membersFromName(String name, |
+ List<InterfaceMirror> defaultTypes) { |
if (!name.contains('.', 0)) { |
if (defaultTypes.isEmpty()) { |
warn('no default type for ${name}'); |
return new Set(); |
} |
- final members = new Set<Member>(); |
+ final members = new Set<MemberMirror>(); |
defaultTypes.forEach((t) { |
- if (t.members.containsKey(name)) members.add(t.members[name]); |
+ MemberMirror member = findMirror(t.declaredMembers(), name); |
+ if (member !== null) { |
+ members.add(member); |
+ } |
}); |
return members; |
} |
@@ -223,10 +248,10 @@ class HtmlDiff { |
var typeName = splitName[0]; |
- final type = dom.types[typeName]; |
+ InterfaceMirror type = findMirror(dom.types(), typeName); |
if (type == null) return new Set(); |
- final member = type.members[splitName[1]]; |
+ MemberMirror member = findMirror(type.declaredMembers(), splitName[1]); |
if (member == null) return new Set(); |
return new Set.from([member]); |