Use ConstExp for storing constants.

tests/compiler/dart2js/backend_dart/sexpr_test.dart

Index: tests/compiler/dart2js/backend_dart/sexpr_test.dart
diff --git a/tests/compiler/dart2js/backend_dart/sexpr_test.dart b/tests/compiler/dart2js/backend_dart/sexpr_test.dart
index a1304633733062e50577378364efb0f509db1486..9ed629c00a24887c6d6ed446983a14e3578be5a5 100644
--- a/tests/compiler/dart2js/backend_dart/sexpr_test.dart
+++ b/tests/compiler/dart2js/backend_dart/sexpr_test.dart
@@ -137,9 +137,14 @@ Future<List<String>> testStringifier(String code,
 /// returns the resulting definitions.
 List<FunctionDefinition> testUnstringifier(List<String> sexprs) {
   return sexprs.map((String sexpr) {
-    final function = new SExpressionUnstringifier().unstringify(sexpr);
-    Expect.isNotNull(function, "Unstringification failed:\n\n$sexpr");
-    return function;
+    try {
+      final function = new SExpressionUnstringifier().unstringify(sexpr);
+      Expect.isNotNull(function, "Unstringification failed:\n\n$sexpr");
+      return function;
+    } catch (e, s) {

[karlklose, 2014/09/25 08:43:25]

Why this change? Is this debug code or is the stacktrace not showing correclty
in the test setup>

[Johnni Winther, 2014/09/29 08:24:44]

On error the unstringifier now throws instead of asserting. This improves the
messaging in case of failures.

+      print('$e\n$s');
+      Expect.fail('Error unstringifying "$sexpr": $e');
+    }
   }).toList();
 }