pkg/docgen: the big refactor

pkg/docgen/lib/src/model_helpers.dart

Index: pkg/docgen/lib/src/model_helpers.dart
diff --git a/pkg/docgen/lib/src/utils.dart b/pkg/docgen/lib/src/model_helpers.dart
similarity index 55%
rename from pkg/docgen/lib/src/utils.dart
rename to pkg/docgen/lib/src/model_helpers.dart
index 8915af48a1c4bc83afa333fea20c0bb508bd2347..95bd31725695b74235e1e49f05fd6bf3e705ac68 100644
--- a/pkg/docgen/lib/src/utils.dart
+++ b/pkg/docgen/lib/src/model_helpers.dart
@@ -2,19 +2,37 @@
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
-library docgen.utils;
+library docgen.model_helpers;
 
-import '../../../../sdk/lib/_internal/compiler/implementation/mirrors/source_mirrors.dart';
+import '../../../../sdk/lib/_internal/compiler/implementation/mirrors/dart2js_mirrors.dart'
+    as dart2js_mirrors;
 import '../../../../sdk/lib/_internal/compiler/implementation/mirrors/mirrors_util.dart'
     as dart2js_util;
+import '../../../../sdk/lib/_internal/compiler/implementation/mirrors/source_mirrors.dart';
 import '../../../../sdk/lib/_internal/libraries.dart';
 
-// HTML escaped version of '<' character.
-const LESS_THAN = '&lt;';
+import 'models.dart';
+import 'package_helpers.dart';
+
+/// Returns a list of meta annotations assocated with a mirror.
+List<Annotation> createAnnotations(DeclarationMirror mirror, Library
+  owningLibrary) {
+  var annotationMirrors = mirror.metadata.where((e) => e is
+      dart2js_mirrors.Dart2JsConstructedConstantMirror);
+  var annotations = [];
+  annotationMirrors.forEach((annotation) {
+    var docgenAnnotation = new Annotation(annotation, owningLibrary);
+    if (!_SKIPPED_ANNOTATIONS.contains(dart2js_util.qualifiedNameOf(
+        docgenAnnotation.mirror))) {
+      annotations.add(docgenAnnotation);
+    }
+  });
+  return annotations;
+}
 
 /// A declaration is private if itself is private, or the owner is private.
 // Issue(12202) - A declaration is public even if it's owner is private.
-bool isHidden(DeclarationMirror mirror) {
+bool isHidden(DeclarationSourceMirror mirror) {
   if (mirror is LibraryMirror) {
     return _isLibraryPrivate(mirror);
   } else if (mirror.owner is LibraryMirror) {
@@ -26,24 +44,6 @@ bool isHidden(DeclarationMirror mirror) {
   }
 }
 
-/// Returns true if a library name starts with an underscore, and false
-/// otherwise.
-///
-/// An example that starts with _ is _js_helper.
-/// An example that contains ._ is dart._collection.dev
-bool _isLibraryPrivate(LibraryMirror mirror) {
-  // This method is needed because LibraryMirror.isPrivate returns `false` all
-  // the time.
-  var sdkLibrary = LIBRARIES[dart2js_util.nameOf(mirror)];
-  if (sdkLibrary != null) {
-    return !sdkLibrary.documented;
-  } else if (dart2js_util.nameOf(mirror).startsWith('_') || dart2js_util.nameOf(
-      mirror).contains('._')) {
-    return true;
-  }
-  return false;
-}
-
 /// Transforms the map by calling toMap on each value in it.
 Map recurseMap(Map inputMap) {
   var outputMap = {};
@@ -65,43 +65,34 @@ Map filterMap(Map map, Function test) {
   return exported;
 }
 
-/// Chunk the provided name into individual parts to be resolved. We take a
-/// simplistic approach to chunking, though, we break at " ", ",", "&lt;"
-/// and ">". All other characters are grouped into the name to be resolved.
-/// As a result, these characters will all be treated as part of the item to
-/// be resolved (aka the * is interpreted literally as a *, not as an
-/// indicator for bold <em>.
-List<String> tokenizeComplexReference(String name) {
-  var tokens = [];
-  var append = false;
-  var index = 0;
-  while (index < name.length) {
-    if (name.indexOf(LESS_THAN, index) == index) {
-      tokens.add(LESS_THAN);
-      append = false;
-      index += LESS_THAN.length;
-    } else if (name[index] == ' ' || name[index] == ',' || name[index] == '>') {
-      tokens.add(name[index]);
-      append = false;
-      index++;
-    } else {
-      if (append) {
-        tokens[tokens.length - 1] = tokens.last + name[index];
-      } else {
-        tokens.add(name[index]);
-        append = true;
-      }
-      index++;
-    }
-  }
-  return tokens;
+/// Read a pubspec and return the library name given a [LibraryMirror].
+String getPackageName(LibraryMirror mirror) {
+  if (mirror.uri.scheme != 'file') return '';
+  var rootdir = getPackageDirectory(mirror);
+  if (rootdir == null) return '';
+  return packageNameFor(rootdir);
 }
 
-typedef DeclarationMirror LookupFunction(DeclarationSourceMirror declaration,
-    String name);
+/// Annotations that we do not display in the viewer.
+const List<String> _SKIPPED_ANNOTATIONS = const [
+  'metadata.DocsEditable', '_js_helper.JSName', '_js_helper.Creates',
+  '_js_helper.Returns'
+];
 
-/// For a given name, determine if we need to resolve it as a qualified name
-/// or a simple name in the source mirors.
-LookupFunction determineLookupFunc(String name) => name.contains('.') ?
-    dart2js_util.lookupQualifiedInScope :
-      (mirror, name) => mirror.lookupInScope(name);
+/// Returns true if a library name starts with an underscore, and false
+/// otherwise.
+///
+/// An example that starts with _ is _js_helper.
+/// An example that contains ._ is dart._collection.dev
+bool _isLibraryPrivate(dart2js_mirrors.Dart2JsLibraryMirror mirror) {
+  // This method is needed because LibraryMirror.isPrivate returns `false` all
+  // the time.
+  var sdkLibrary = LIBRARIES[dart2js_util.nameOf(mirror)];
+  if (sdkLibrary != null) {
+    return !sdkLibrary.documented;
+  } else if (dart2js_util.nameOf(mirror).startsWith('_') || dart2js_util.nameOf(
+      mirror).contains('._')) {
+    return true;
+  }
+  return false;
+}