Tag closures with their types

lib/src/js/nodes.dart

Index: lib/src/js/nodes.dart
diff --git a/lib/src/js/nodes.dart b/lib/src/js/nodes.dart
index edad282582a57ecb9f196b444fd4e11ef46b39c7..f66db108dc198855e33cc0928e00cd446ad347d7 100644
--- a/lib/src/js/nodes.dart
+++ b/lib/src/js/nodes.dart
@@ -1134,11 +1134,12 @@ class ArrayHole extends Expression {
 
 class ObjectInitializer extends Expression {
   final List<Property> properties;
+  final bool vertical;

[Jennifer Messerly, 2015/05/19 18:19:01]

This could be computed:

bool _vertical;
bool get vertical {
  if (_vertical == null) {
    _vertical = properties.any((p) => p.value is FunctionExpression;
  }
  return _vertical;
}

And then that simplifies Printer.

Not sure I like that better or worse, just throwing it out as an idea to see
what you think :)

Regardless, maybe add a doc comment:

/**
 * If set to true, forces a vertical layout when using the [Printer].
 * Otherwise, layout will be vertical if and only if any [properties]
 * are [FunctionExpression]s.
 */

[Leaf, 2015/05/19 22:31:13]

Done.

 
   /**
    * Constructs a new object-initializer containing the given [properties].
    */
-  ObjectInitializer(this.properties);
+  ObjectInitializer(this.properties, {this.vertical: false});
 
   accept(NodeVisitor visitor) => visitor.visitObjectInitializer(this);