Nicer pretty printing of jsAst object literals.

sdk/lib/_internal/compiler/implementation/js/printer.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;
 
 class Printer implements NodeVisitor {
   final bool shouldCompressOutput;
   leg.Compiler compiler;
   leg.CodeBuffer outBuffer;
@@ skipping 784 matching lines @@
   }
 
   visitArrayElement(ArrayElement node) {
     throw "Unreachable";
   }
 
   visitObjectInitializer(ObjectInitializer node) {
     // Print all the properties on one line until we see a function-valued
     // property.  Ideally, we would use a proper pretty-printer to make the
     // decision based on layout.
-    bool onePerLine = false;
     List<Property> properties = node.properties;
     out("{");
     ++indentLevel;
     for (int i = 0; i < properties.length; i++) {
       Expression value = properties[i].value;
-      if (value is Fun || value is NamedFunction) onePerLine = true;
       if (i != 0) {
         out(",");
-        if (!onePerLine) spaceOut();
+        if (node.isOneLiner) spaceOut();
       }
-      if (onePerLine) {
+      if (!node.isOneLiner) {
         forceLine();
         indent();
       }
       visitProperty(properties[i]);
     }
     --indentLevel;
-    if (onePerLine) lineOut();
+    if (!node.isOneLiner && !properties.isEmpty) {
+      lineOut();
+      indent();
+    }
     out("}");
   }
 
   visitProperty(Property node) {
     if (node.name is LiteralString) {
       LiteralString nameString = node.name;
       String name = nameString.value;
       if (isValidJavaScriptId(name)) {
         out(name.substring(1, name.length - 1));
       } else {
@@ skipping 307 matching lines @@
         codes.add(nthLetter((n ~/ nameSpaceSize) % LETTERS));
       }
       codes.add(charCodes.$0 + digit);
       newName = new String.fromCharCodes(codes);
     }
     assert(new RegExp(r'[a-zA-Z][a-zA-Z0-9]*').hasMatch(newName));
     maps.last[oldName] = newName;
     return newName;
   }
 }