Handle symbol literals

lib/src/codegen/js_codegen.dart

Index: lib/src/codegen/js_codegen.dart
diff --git a/lib/src/codegen/js_codegen.dart b/lib/src/codegen/js_codegen.dart
index afbeb695f176f000612fd3b1258acf8bf09396d4..786c2dc4565e98dd1764c6c3fa6f64ba13113013 100644
--- a/lib/src/codegen/js_codegen.dart
+++ b/lib/src/codegen/js_codegen.dart
@@ -2053,6 +2053,15 @@ class JSCodegenVisitor extends GeneralizingAstVisitor with ConversionVisitor {
   visitNullLiteral(NullLiteral node) => new JS.LiteralNull();
 
   @override
+  visitSymbolLiteral(SymbolLiteral node) {
+    // TODO(vsm): When we canonicalize, we need to treat private symbols
+    // correctly.
+    var name = js.escapedString(node.components.join('.'));
+    var symbol = js.call('new _internal.Symbol(#)', name);

[Jennifer Messerly, 2015/04/21 21:23:35]

this should be core.Symbol right?
(the impl of core.Symbol currently redirects to _internal via factory, but
that's in the patch file, we might very well want to change it...)

+    return js.commentExpression('Unimplemented const', symbol);
+  }
+
+  @override
   visitListLiteral(ListLiteral node) {
     JS.Expression list = new JS.ArrayInitializer(_visitList(node.elements));