fixes #6, refactor to use AnalysisError/Listener throughout dev_compiler

lib/src/checker/checker.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 dev_compiler.src.checker.checker;
 
 import 'package:analyzer/analyzer.dart';
 import 'package:analyzer/src/generated/ast.dart';
 import 'package:analyzer/src/generated/element.dart';
 import 'package:analyzer/src/generated/scanner.dart' show Token, TokenType;
-import 'package:logging/logging.dart' as logger;
 
 import 'package:dev_compiler/src/info.dart';
-import 'package:dev_compiler/src/report.dart' show CheckerReporter;
 import 'package:dev_compiler/src/utils.dart' show getMemberType;
 import 'package:dev_compiler/strong_mode.dart' show StrongModeOptions;
 import 'rules.dart';
 
 /// Checks for overriding declarations of fields and methods. This is used to
 /// check overrides between classes and superclasses, interfaces, and mixin
 /// applications.
 class _OverrideChecker {
   bool _failure = false;
   final TypeRules _rules;
-  final CheckerReporter _reporter;
+  final AnalysisErrorListener _reporter;
   final bool _inferFromOverrides;
   _OverrideChecker(this._rules, this._reporter, StrongModeOptions options)
       : _inferFromOverrides = options.inferFromOverrides;
 
   void check(ClassDeclaration node) {
     if (node.element.type.isObject) return;
     _checkSuperOverrides(node);
     _checkMixinApplicationOverrides(node);
     _checkAllInterfaceOverrides(node);
   }
@@ skipping 284 matching lines @@
     // node is a MethodDeclaration whenever getters and setters are
     // declared explicitly. Setters declared from a field will have the
     // correct return type, so we don't need to check that separately.
     return node is MethodDeclaration &&
         node.returnType == null &&
         _rules.isFunctionSubTypeOf(subType, baseType, ignoreReturn: true);
   }
 
   void _recordMessage(StaticInfo info) {
     if (info == null) return;
-    if (info.level >= logger.Level.SEVERE) _failure = true;
-    _reporter.log(info);
+    var error = info.toAnalysisError();
+    if (error.errorCode.errorSeverity == ErrorSeverity.ERROR) _failure = true;
+    _reporter.onError(error);
   }
 }
 
 /// Checks the body of functions and properties.
 class CodeChecker extends RecursiveAstVisitor {
   final TypeRules _rules;
-  final CheckerReporter _reporter;
+  final AnalysisErrorListener _reporter;
   final _OverrideChecker _overrideChecker;
   bool _failure = false;
   bool get failure => _failure || _overrideChecker._failure;
 
-  CodeChecker(
-      TypeRules rules, CheckerReporter reporter, StrongModeOptions options)
+  CodeChecker(TypeRules rules, AnalysisErrorListener reporter,
+      StrongModeOptions options)
       : _rules = rules,
         _reporter = reporter,
         _overrideChecker = new _OverrideChecker(rules, reporter, options);
 
   @override
   void visitCompilationUnit(CompilationUnit unit) {
     void report(Expression expr) {
-      _reporter.log(new MissingTypeError(expr));
+      _reporter.onError(new MissingTypeError(expr).toAnalysisError());
     }
     var callback = _rules.reportMissingType;
     _rules.reportMissingType = report;
     unit.visitChildren(this);
     _rules.reportMissingType = callback;
   }
 
   @override
   void visitComment(Comment node) {
     // skip, no need to do typechecking inside comments (they may contain
@@ skipping 618 matching lines @@
       // Check the rhs type
       if (staticInfo is! CoercionInfo) {
         var paramType = paramTypes.first;
         staticInfo = _rules.checkAssignment(expr.rightHandSide, paramType);
         _recordMessage(staticInfo);
       }
     }
   }
 
   void _recordDynamicInvoke(AstNode node, AstNode target) {
-    var dinvoke = new DynamicInvoke(_rules, node);
-    _reporter.log(dinvoke);
+    _reporter.onError(new DynamicInvoke(_rules, node).toAnalysisError());
     // TODO(jmesserly): we may eventually want to record if the whole operation
     // (node) was dynamic, rather than the target, but this is an easier fit
     // with what we used to do.
     DynamicInvoke.set(target, true);
   }
 
   void _recordMessage(StaticInfo info) {
     if (info == null) return;
-    if (info.level >= logger.Level.SEVERE) _failure = true;
-    _reporter.log(info);
+    var error = info.toAnalysisError();
+    if (error.errorCode.errorSeverity == ErrorSeverity.ERROR) _failure = true;
+    _reporter.onError(error);
+
     if (info is CoercionInfo) {
       assert(CoercionInfo.get(info.node) == null);
       CoercionInfo.set(info.node, info);
     }
   }
 }