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

Unified Diff: compiler/javatests/com/google/dart/compiler/CompilerTestCase.java

Issue 9138023: More setting of symbols on identifiers (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Changes many assertEquals() to assertSame() Created 8 years, 11 months 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: compiler/javatests/com/google/dart/compiler/CompilerTestCase.java
diff --git a/compiler/javatests/com/google/dart/compiler/CompilerTestCase.java b/compiler/javatests/com/google/dart/compiler/CompilerTestCase.java
index 8ae0ad72112067deb8d5551a04fc69ed626166ea..2be9940d8367a018d1068d425aae755946f1faab 100644
--- a/compiler/javatests/com/google/dart/compiler/CompilerTestCase.java
+++ b/compiler/javatests/com/google/dart/compiler/CompilerTestCase.java
@@ -1,4 +1,4 @@
-// Copyright (c) 2011, the Dart project authors. Please see the AUTHORS file
+// 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.
@@ -189,6 +189,10 @@ public abstract class CompilerTestCase extends TestCase {
DartArtifactProvider provider = new MockArtifactProvider();
result.setLibraryUnitResult(DartCompiler.analyzeLibrary(lib, testUnits, config, provider,
result));
+ // TODO(zundel): One day, we want all AST nodes that are identifiers to point to
+ // elements if they are resolved. Uncommenting this line helps track missing elements
+ // down.
+ // ResolverAuditVisitor.exec(unit);
return result;
}

Powered by Google App Engine
This is Rietveld 408576698