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

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

Issue 221873002: Compute frontend/backend specific constants. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 9 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: tests/compiler/dart2js/mirrors_used_test.dart
diff --git a/tests/compiler/dart2js/mirrors_used_test.dart b/tests/compiler/dart2js/mirrors_used_test.dart
index 846c7f9d95ef88c7aeac41741a48f4219041415e..3b7b129dd0c3b9817526ce8ed1076a03184b2e90 100644
--- a/tests/compiler/dart2js/mirrors_used_test.dart
+++ b/tests/compiler/dart2js/mirrors_used_test.dart
@@ -101,19 +101,19 @@ void main() {
// 1. The constructed constant for 'MirrorsUsed'.
Expect.isTrue(compiler.backend.metadataConstants.length >= 1);
+ Set<Constant> compiledConstants =
+ compiler.backend.constantHandler.compiledConstants;
// Make sure that most of the metadata constants aren't included in the
// generated code.
for (var dependency in compiler.backend.metadataConstants) {
Constant constant = dependency.constant;
- Expect.isFalse(
- compiler.constantHandler.compiledConstants.contains(constant),
- '$constant');
+ Expect.isFalse(compiledConstants.contains(constant), '$constant');
}
// The type literal 'Foo' is both used as metadata, and as a plain value in
// the program. Make sure that it isn't duplicated.
int fooConstantCount = 0;
- for (Constant constant in compiler.constantHandler.compiledConstants) {
+ for (Constant constant in compiledConstants) {
if (constant is TypeConstant && '${constant.representedType}' == 'Foo') {
fooConstantCount++;
}

Powered by Google App Engine
This is Rietveld 408576698