Remove Compiler/JavaScriptBackend from metadata_collector

pkg/compiler/lib/src/js/js.dart

 // Copyright (c) 2015, 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 js;
 
 import 'package:js_ast/js_ast.dart';
 
 import '../common.dart';
-import '../compiler.dart' show Compiler;
+import '../options.dart';
 import '../dump_info.dart' show DumpInfoTask;
 import '../io/code_output.dart' show CodeBuffer;
 import '../js_emitter/js_emitter.dart' show USE_LAZY_EMITTER;
 import 'js_source_mapping.dart';
 
 export 'package:js_ast/js_ast.dart';
 
-String prettyPrint(Node node, Compiler compiler,
+String prettyPrint(Node node, CompilerOptions compilerOptions,
     {bool allowVariableMinification: true,
     Renamer renamerForNames: JavaScriptPrintingOptions.identityRenamer}) {
   // TODO(johnniwinther): Do we need all the options here?
   JavaScriptPrintingOptions options = new JavaScriptPrintingOptions(
-      shouldCompressOutput: compiler.options.enableMinification,
+      shouldCompressOutput: compilerOptions.enableMinification,
       minifyLocalVariables: allowVariableMinification,
       preferSemicolonToNewlineInMinifiedOutput: USE_LAZY_EMITTER,
       renamerForNames: renamerForNames);
   SimpleJavaScriptPrintingContext context =
       new SimpleJavaScriptPrintingContext();
   Printer printer = new Printer(options, context);
   printer.visit(node);
   return context.getText();
 }
 
-CodeBuffer createCodeBuffer(Node node, Compiler compiler,
+CodeBuffer createCodeBuffer(Node node, CompilerOptions compilerOptions,
+    JavaScriptSourceInformationStrategy sourceInformationStrategy,
     {DumpInfoTask monitor,
     bool allowVariableMinification: true,
     Renamer renamerForNames: JavaScriptPrintingOptions.identityRenamer}) {
-  JavaScriptSourceInformationStrategy sourceInformationFactory =
-      compiler.backend.sourceInformationStrategy;
   JavaScriptPrintingOptions options = new JavaScriptPrintingOptions(
-      shouldCompressOutput: compiler.options.enableMinification,
+      shouldCompressOutput: compilerOptions.enableMinification,
       minifyLocalVariables: allowVariableMinification,
       preferSemicolonToNewlineInMinifiedOutput: USE_LAZY_EMITTER,
       renamerForNames: renamerForNames);
   CodeBuffer outBuffer = new CodeBuffer();
   SourceInformationProcessor sourceInformationProcessor =
-      sourceInformationFactory.createProcessor(
+      sourceInformationStrategy.createProcessor(
           new SourceMapperProviderImpl(outBuffer),
           const SourceInformationReader());
   Dart2JSJavaScriptPrintingContext context =
       new Dart2JSJavaScriptPrintingContext(
-          compiler.reporter, monitor, outBuffer, sourceInformationProcessor);
+          monitor, outBuffer, sourceInformationProcessor);
   Printer printer = new Printer(options, context);
   printer.visit(node);
   sourceInformationProcessor.process(node, outBuffer);
   return outBuffer;
 }
 
 class Dart2JSJavaScriptPrintingContext implements JavaScriptPrintingContext {
-  final DiagnosticReporter reporter;
   final DumpInfoTask monitor;
   final CodeBuffer outBuffer;
   final CodePositionListener codePositionListener;
 
   Dart2JSJavaScriptPrintingContext(
-      this.reporter, this.monitor, this.outBuffer, this.codePositionListener);
+      this.monitor, this.outBuffer, this.codePositionListener);
 
   @override
   void error(String message) {
-    reporter.internalError(NO_LOCATION_SPANNABLE, message);
+    throw new SpannableAssertionFailure(NO_LOCATION_SPANNABLE, message);
   }
 
   @override
   void emit(String string) {
     outBuffer.add(string);
   }
 
   @override
   void enterNode(Node node, int startPosition) {
     codePositionListener.onStartPosition(node, startPosition);
@@ skipping 45 matching lines @@
 }
 
 /// Represents the LiteralString resulting from unparsing [expression]. The
 /// actual unparsing is done on demand when requesting the [value] of this
 /// node.
 ///
 /// This is used when generated code needs to be represented as a string,
 /// for example by the lazy emitter or when generating code generators.
 class UnparsedNode extends DeferredString implements AstContainer {
   final Node tree;
-  final Compiler _compiler;
+  final CompilerOptions _compilerOptions;
   final bool _protectForEval;
   LiteralString _cachedLiteral;
 
   Iterable<Node> get containedNodes => [tree];
 
   /// A [js.Literal] that represents the string result of unparsing [ast].
   ///
   /// When its string [value] is requested, the node pretty-prints the given
   /// [ast] and, if [protectForEval] is true, wraps the resulting string in
   /// parenthesis. The result is also escaped.
-  UnparsedNode(this.tree, this._compiler, this._protectForEval);
+  UnparsedNode(this.tree, this._compilerOptions, this._protectForEval);
 
   LiteralString get _literal {
     if (_cachedLiteral == null) {
-      String text = prettyPrint(tree, _compiler);
+      String text = prettyPrint(tree, _compilerOptions);
       if (_protectForEval) {
         if (tree is Fun) text = '($text)';
         if (tree is LiteralExpression) {
           LiteralExpression literalExpression = tree;
           String template = literalExpression.template;
           if (template.startsWith("function ") || template.startsWith("{")) {
             text = '($text)';
           }
         }
       }
@@ skipping 27 matching lines @@
   }
   if (node is PropertyAccess) {
     PropertyAccess access = node;
     if (access.receiver is InterpolatedExpression) {
       InterpolatedExpression hole = access.receiver;
       return hole.isPositional && hole.nameOrPosition == 0;
     }
   }
   return false;
 }