refactor emitMemberName to be used more consistently

lib/src/js/nodes.dart

 // 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.
 
 part of js_ast;
 
 abstract class NodeVisitor<T> {
   T visitProgram(Program node);
 
   T visitBlock(Block node);
@@ skipping 1060 matching lines @@
   /**
    * Constructs a LiteralString from a string value.
    *
    * The constructor does not add the required quotes.  If [value] is not
    * surrounded by quotes and property escaped, the resulting object is invalid
    * as a JS value.
    *
    * TODO(sra): Introduce variants for known valid strings that don't allocate a
    * new string just to add quotes.
    */
-  LiteralString(this.value);
+  LiteralString(this.value) {
+    assert(value != '"_nextCallback="');

[Jacob, 2015/04/13 23:19:36]

this seems like an oddly specific assert.

[Jennifer Messerly, 2015/04/14 21:21:00]

eeeep. Good catch! I didn't follow my own convention for marking debug code :\

+  }
 
   accept(NodeVisitor visitor) => visitor.visitLiteralString(this);
   LiteralString _clone() => new LiteralString(value);
 }
 
 class LiteralNumber extends Literal {
   final String value;  // Must be a valid JavaScript number literal.
 
   LiteralNumber(this.value);
 
@@ skipping 339 matching lines @@
   final Expression expression;
 
   CommentExpression(this.comment, this.expression);
 
   int get precedenceLevel => PRIMARY;
   accept(NodeVisitor visitor) => visitor.visitCommentExpression(this);
   CommentExpression _clone() => new CommentExpression(comment, expression);
 
   void visitChildren(NodeVisitor visitor) => expression.accept(visitor);
 }