Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(431)

Unified Diff: tests/compiler/dart2js/type_checker_test.dart

Issue 1415463022: Use DiagnosticMessage in MockCompiler. (Closed) Base URL: https://github.com/dart-lang/sdk.git@master
Patch Set: Created 5 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: tests/compiler/dart2js/type_checker_test.dart
diff --git a/tests/compiler/dart2js/type_checker_test.dart b/tests/compiler/dart2js/type_checker_test.dart
index ff7638d9326338705c49a5573ca8c06b6d0dfbf7..d0e7db555afb6f19413bc9471aa237845d343935 100644
--- a/tests/compiler/dart2js/type_checker_test.dart
+++ b/tests/compiler/dart2js/type_checker_test.dart
@@ -2524,9 +2524,10 @@ analyzeTopLevel(String text, [expectedWarnings]) {
// Type check last class declaration or member.
TypeCheckerVisitor checker =
new TypeCheckerVisitor(compiler, mapping, compiler.types);
- compiler.clearMessages();
+ DiagnosticCollector collector = compiler.diagnosticCollector;
+ collector.clear();
checker.analyze(node);
- compareWarningKinds(text, expectedWarnings, compiler.warnings);
+ compareWarningKinds(text, expectedWarnings, collector.warnings);
compiler.diagnosticHandler = null;
});
@@ -2562,25 +2563,23 @@ analyze(MockCompiler compiler,
compiler.enqueuer.resolution.emptyDeferredTaskQueue();
TypeCheckerVisitor checker = new TypeCheckerVisitor(
compiler, elements, compiler.types);
- compiler.clearMessages();
+ DiagnosticCollector collector = compiler.diagnosticCollector;
+ collector.clear();
checker.analyze(node);
if (flushDeferred) {
compiler.enqueuer.resolution.emptyDeferredTaskQueue();
}
- compareWarningKinds(text, warnings, compiler.warnings);
- compareWarningKinds(text, errors, compiler.errors);
- if (hints != null) compareWarningKinds(text, hints, compiler.hints);
- if (infos != null) compareWarningKinds(text, infos, compiler.infos);
+ compareWarningKinds(text, warnings, collector.warnings);
+ compareWarningKinds(text, errors, collector.errors);
+ if (hints != null) compareWarningKinds(text, hints, collector.hints);
+ if (infos != null) compareWarningKinds(text, infos, collector.infos);
compiler.diagnosticHandler = null;
}
void generateOutput(MockCompiler compiler, String text) {
- for (WarningMessage message in compiler.warnings) {
- Node node = message.node;
- var beginToken = node.getBeginToken();
- var endToken = node.getEndToken();
- int begin = beginToken.charOffset;
- int end = endToken.charOffset + endToken.charCount;
+ for (CollectedMessage message in compiler.diagnosticCollector.warnings) {
+ int begin = message.begin;
+ int end = message.end;
SourceFile sourceFile = new StringSourceFile.fromName('analysis', text);
print(sourceFile.getLocationMessage(message.message.toString(),
begin, end));
@@ -2608,9 +2607,10 @@ analyzeIn(MockCompiler compiler,
TreeElements elements = compiler.resolveNodeStatement(node, element);
TypeCheckerVisitor checker = new TypeCheckerVisitor(
compiler, elements, compiler.types);
- compiler.clearMessages();
+ DiagnosticCollector collector = compiler.diagnosticCollector;
+ collector.clear();
checker.analyze(node);
generateOutput(compiler, text);
- compareWarningKinds(text, warnings, compiler.warnings);
- compareWarningKinds(text, hints, compiler.hints);
+ compareWarningKinds(text, warnings, collector.warnings);
+ compareWarningKinds(text, hints, collector.hints);
}

Powered by Google App Engine
This is Rietveld 408576698