Support full dart2js output for dartdoc/apidoc.

sdk/lib/_internal/compiler/implementation/mirrors/dart2js_mirror.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // 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 mirrors_dart2js;
 
 import 'dart:async';
 import 'dart:collection' show LinkedHashMap;
 import 'dart:io';
 import 'dart:uri';
 
-import '../../compiler.dart' as diagnostics;
+import '../../compiler.dart' as api;
 import '../elements/elements.dart';
 import '../resolution/resolution.dart' show ResolverTask, ResolverVisitor;
 import '../apiimpl.dart' show Compiler;
 import '../scanner/scannerlib.dart' hide SourceString;
 import '../ssa/ssa.dart';
 import '../dart2jslib.dart' hide Compiler;
 import '../dart_types.dart';
 import '../filenames.dart';
 import '../source_file.dart';
 import '../tree/tree.dart';
@@ skipping 54 matching lines @@
   } else if (type is FunctionType) {
     return new Dart2JsFunctionTypeMirror(system, type, functionSignature);
   } else if (type is VoidType) {
     return new Dart2JsVoidMirror(system, type);
   } else if (type is TypedefType) {
     return new Dart2JsTypedefMirror(system, type);
   } else if (type is MalformedType) {
     // TODO(johnniwinther): We need a mirror on malformed types.
     return system.dynamicType;
   }
-  _diagnosticListener.internalError(
-      "Unexpected type $type of kind ${type.kind}");
   system.compiler.internalError("Unexpected type $type of kind ${type.kind}");
 }
 
 Collection<Dart2JsMemberMirror> _convertElementMemberToMemberMirrors(
     Dart2JsContainerMirror library, Element element) {
   if (element.isSynthesized) {
     return const <Dart2JsMemberMirror>[];
   } else if (element is VariableElement) {
     return <Dart2JsMemberMirror>[new Dart2JsFieldMirror(library, element)];
   } else if (element is FunctionElement) {
@@ skipping 87 matching lines @@
     'xor': '^',
     'or': '|',
   };
   String newName = mapping[name];
   if (newName == null) {
     throw new Exception('Unhandled operator name: $name');
   }
   return newName;
 }
 
-DiagnosticListener get _diagnosticListener {
-  return const Dart2JsDiagnosticListener();
-}
-
-class Dart2JsDiagnosticListener implements DiagnosticListener {
-  const Dart2JsDiagnosticListener();
-
-  void cancel(String reason, {node, token, instruction, element}) {
-    print(reason);
-  }
-
-  void log(message) {
-    print(message);
-  }
-
-  void internalError(String message,
-                     {Node node, Token token, HInstruction instruction,
-                      Element element}) {
-    cancel('Internal error: $message', node: node, token: token,
-           instruction: instruction, element: element);
-  }
-
-  void internalErrorOnElement(Element element, String message) {
-    internalError(message, element: element);
-  }
-
-  SourceSpan spanFromSpannable(Node node, [Uri uri]) {
-    // TODO(johnniwinther): implement this.
-    throw 'unimplemented';
-  }
-
-  void reportMessage(SourceSpan span, Diagnostic message,
-                     diagnostics.Diagnostic kind) {
-    // TODO(johnniwinther): implement this.
-    throw 'unimplemented';
-  }
-
-  bool onDeprecatedFeature(Spannable span, String feature) {
-    // TODO(johnniwinther): implement this?
-    throw 'unimplemented';
-  }
-}
-
 //------------------------------------------------------------------------------
 // Compilation implementation
 //------------------------------------------------------------------------------
 
-// TODO(johnniwinther): Support client configurable handlers/providers.
-class Dart2JsCompilation implements Compilation {
-  Compiler _compiler;
-  final Uri cwd;
-  final SourceFileProvider provider;
+// TODO(johnniwinther): Support client configurable providers.
 
-  Dart2JsCompilation(Path script, Path libraryRoot,
-                     [Path packageRoot, List<String> opts = const <String>[]])
-      : cwd = getCurrentDirectory(),
-        provider = new SourceFileProvider() {
-    var handler = new FormattingDiagnosticHandler(provider);
-    var libraryUri = cwd.resolve('${libraryRoot}/');
-    var packageUri;
-    if (packageRoot != null) {
-      packageUri = cwd.resolve('${packageRoot}/');
-    } else {
-      packageUri = libraryUri;
+/**
+ * Returns a future that completes to a non-null String when [script]
+ * has been successfully compiled.

[ahe, 2013/03/05 14:48:31]

Add TODO and explain this deprecated but here to support Path which are uses
throughout dartdoc.

[Johnni Winther, 2013/03/06 08:13:11]

Done.

+ */
+Future<String> compile(Path script,
+                       Path libraryRoot,
+                       {Path packageRoot,
+                        List<String> options: const <String>[],
+                        api.DiagnosticHandler diagnosticHandler}) {
+  Uri cwd = getCurrentDirectory();
+  SourceFileProvider provider = new SourceFileProvider();
+  if (diagnosticHandler == null) {
+    diagnosticHandler =
+        new FormattingDiagnosticHandler(provider).diagnosticHandler;
+  }
+  Uri scriptUri = cwd.resolve(script.toString());
+  Uri libraryUri = cwd.resolve('${libraryRoot}/');
+  Uri packageUri = null;
+  if (packageRoot != null) {
+    packageUri = cwd.resolve('${packageRoot}/');
+  }
+  return api.compile(scriptUri, libraryUri, packageUri,
+      provider.readStringFromUri, diagnosticHandler, options);
+}
+
+/**
+ * Analyzes set of libraries and provides a mirror system which can be used for
+ * static inspection of the source code.
+ */
+Future<MirrorSystem> analyze(List<Path> libraries,
+                             Path libraryRoot,
+                             {Path packageRoot,
+                              List<String> options: const <String>[],
+                              api.DiagnosticHandler diagnosticHandler}) {
+  Uri cwd = getCurrentDirectory();
+  SourceFileProvider provider = new SourceFileProvider();
+  if (diagnosticHandler == null) {
+    diagnosticHandler =
+        new FormattingDiagnosticHandler(provider).diagnosticHandler;
+  }
+  Uri libraryUri = cwd.resolve('${libraryRoot}/');
+  Uri packageUri = null;
+  if (packageRoot != null) {
+    packageUri = cwd.resolve('${packageRoot}/');
+  }
+  options = new List<String>.from(options, growable: true);

[ahe, 2013/03/05 14:48:31]

SIGH!!!

[Johnni Winther, 2013/03/06 08:13:11]

Done.

+  options.add('--analyze-only');
+  options.add('--analyze-signatures-only');
+  options.add('--analyze-all');
+
+  bool compilationFailed = false;
+  void internalDiagnosticHandler(Uri uri, int begin, int end,
+                                 String message, api.Diagnostic kind) {
+    if (kind == api.Diagnostic.ERROR ||
+        kind == api.Diagnostic.CRASH) {
+      compilationFailed = true;
     }
-    _compiler = new Compiler(provider.readStringFromUri,
-                                 null,
-                                 handler.diagnosticHandler,
-                                 libraryUri, packageUri, opts);
-    var scriptUri = cwd.resolve(script.toString());
-    // TODO(johnniwinther): Detect file not found
-    _compiler.run(scriptUri);
+    diagnosticHandler(uri, begin, end, message, kind);
   }
 
-  Dart2JsCompilation.library(List<Path> libraries, Path libraryRoot,
-                     [Path packageRoot, List<String> opts = const <String>[]])
-      : cwd = getCurrentDirectory(),
-        provider = new SourceFileProvider() {
-    var libraryUri = cwd.resolve('${libraryRoot}/');
-    var packageUri;
-    if (packageRoot != null) {
-      packageUri = cwd.resolve('${packageRoot}/');
-    } else {
-      packageUri = libraryUri;
-    }
-    opts = new List<String>.from(opts);
-    opts.add('--analyze-only');
-    opts.add('--analyze-signatures-only');
-    opts.add('--analyze-all');
-    _compiler = new Compiler(provider.readStringFromUri,
-                                 null,
-                                 silentDiagnosticHandler,
-                                 libraryUri, packageUri, opts);
-    var librariesUri = <Uri>[];
-    for (Path library in libraries) {
-      librariesUri.add(cwd.resolve(library.toString()));
-      // TODO(johnniwinther): Detect file not found
-    }
-    _compiler.librariesToAnalyzeWhenRun = librariesUri;
-    _compiler.run(null);
+  // TODO(johnniwinther): Extract the following as an [analyze] method in
+  // [:compiler/compiler.dart:].
+  Compiler compiler = new Compiler(provider.readStringFromUri,
+                                   null,
+                                   internalDiagnosticHandler,
+                                   libraryUri, packageUri, options);
+  Completer<MirrorSystem> result = new Completer<MirrorSystem>();

[ahe, 2013/03/05 14:48:31]

I think you can use "new Future.immediate" and "new Future.immediateError" here.

[Johnni Winther, 2013/03/06 08:13:11]

Done.

+  List<Uri> librariesUri = <Uri>[];
+  for (Path library in libraries) {
+    librariesUri.add(cwd.resolve(library.toString()));
   }
-
-  MirrorSystem get mirrors => new Dart2JsMirrorSystem(_compiler);
-
-  Future<String> compileToJavaScript() =>
-      new Future<String>.immediate(_compiler.assembledCode);
+  compiler.librariesToAnalyzeWhenRun = librariesUri;
+  bool success = compiler.run(null);
+  if (success && !compilationFailed) {
+    result.complete(new Dart2JsMirrorSystem(compiler));
+  } else {
+    result.completeError('Failed to create mirror system.');
+  }
+  return result.future;
 }
 
-
 //------------------------------------------------------------------------------
 // Dart2Js specific extensions of mirror interfaces
 //------------------------------------------------------------------------------
 
 abstract class Dart2JsMirror implements Mirror {
   Dart2JsMirrorSystem get mirrors;
 }
 
 abstract class Dart2JsDeclarationMirror extends Dart2JsMirror
     implements DeclarationMirror {
@@ skipping 1419 matching lines @@
       return new Future.immediate(
           new Dart2JsStringConstantMirror.fromString(mirrors, text));
     } else if (fieldName == 'trimmedText') {
       return new Future.immediate(
           new Dart2JsStringConstantMirror.fromString(mirrors, trimmedText));
     }
     // TODO(johnniwinther): Which exception/error should be thrown here?
     throw new UnsupportedError('InstanceMirror does not have a reflectee');
   }
 }