Improve the RenderObject.toString() logic so that there's not blank lines everywhere and to general…

sky/sdk/lib/framework/rendering/box.dart

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 import 'dart:math' as math;
 import 'dart:sky' as sky;
 import 'dart:typed_data';
 import 'object.dart';
 import 'package:vector_math/vector_math.dart';
 import 'package:sky/framework/net/image_cache.dart' as image_cache;
@@ skipping 132 matching lines @@
   }
 
   bool hitTest(HitTestResult result, { sky.Point position }) {
     hitTestChildren(result, position: position);
     result.add(this);
     return true;
   }
   void hitTestChildren(HitTestResult result, { sky.Point position }) { }
 
   sky.Size size = new sky.Size(0.0, 0.0);
+
+  String debugDescribeSettings(String prefix) => '${super.debugDescribeSettings(prefix)}${prefix}size: ${size}\n';
 }
 
 abstract class RenderProxyBox extends RenderBox with RenderObjectWithChildMixin<RenderBox> {
   RenderProxyBox(RenderBox child) {
     this.child = child;
   }
 
   sky.Size getIntrinsicDimensions(BoxConstraints constraints) {
     if (child != null)
       return child.getIntrinsicDimensions(constraints);
@@ skipping 45 matching lines @@
   sky.Size getIntrinsicDimensions(BoxConstraints constraints) {
     return constraints.constrain(_desiredSize);
   }
 
   void performLayout() {
     size = constraints.constrain(_desiredSize);
     if (child != null)
       child.layout(new BoxConstraints.tight(size));
   }
 
-  String debugDescribeSettings(String prefix) => '${prefix}desiredSize: ${desiredSize}';
+  String debugDescribeSettings(String prefix) => '${super.debugDescribeSettings(prefix)}${prefix}desiredSize: ${desiredSize}\n';
 }
 
 class RenderClip extends RenderProxyBox {
   RenderClip({ RenderBox child }) : super(child);
 
   void paint(RenderObjectDisplayList canvas) {
     if (child != null) {
       canvas.save();
       canvas.clipRect(new sky.Rect.fromSize(size));
       child.paint(canvas);
@@ skipping 52 matching lines @@
     if (child != null) {
       assert(child.parentData is BoxParentData);
       sky.Rect childBounds = new sky.Rect.fromPointAndSize(child.parentData.position, child.size);
       if (childBounds.contains(position)) {
         child.hitTest(result, position: new sky.Point(position.x - child.parentData.position.x,
                                                       position.y - child.parentData.position.y));
       }
     }
   }
 
-  String debugDescribeSettings(String prefix) => '${prefix}padding: ${padding}';
+  String debugDescribeSettings(String prefix) => '${super.debugDescribeSettings(prefix)}${prefix}padding: ${padding}\n';
 }
 
 class RenderImage extends RenderBox {
 
   RenderImage(String url, sky.Size dimensions) {
     requestedSize = dimensions;
     src = url;
   }
 
   sky.Image _image;
@@ skipping 55 matching lines @@
       double heightScale = size.height / _image.height;
       canvas.save();
       canvas.scale(widthScale, heightScale);
     }
     sky.Paint paint = new sky.Paint();
     canvas.drawImage(_image, 0.0, 0.0, paint);
     if (needsScale)
       canvas.restore();
   }
 
-  String debugDescribeSettings(String prefix) => '${prefix}url: ${src}\n${prefix}dimensions: ${requestedSize}';
+  String debugDescribeSettings(String prefix) => '${super.debugDescribeSettings(prefix)}${prefix}url: ${src}\n${prefix}dimensions: ${requestedSize}\n';
 }
 
 class BorderSide {
   const BorderSide({
     this.color: const sky.Color(0xFF000000),
     this.width: 1.0
   });
   final sky.Color color;
   final double width;
 
@@ skipping 128 matching lines @@
       path.lineTo(_decoration.border.left.width, size.height - _decoration.border.bottom.width);
       path.lineTo(_decoration.border.left.width, _decoration.border.top.width);
       path.lineTo(0.0,0.0);
       path.close();
       canvas.drawPath(path, paint);
     }
 
     super.paint(canvas);
   }
 
-  String debugDescribeSettings(String prefix) => '${prefix}decoration:\n${decoration.toString(prefix + "  ")}';
+  String debugDescribeSettings(String prefix) => '${super.debugDescribeSettings(prefix)}${prefix}decoration:\n${decoration.toString(prefix + "  ")}\n';
 }
 
 class RenderTransform extends RenderProxyBox {
   RenderTransform({
     Matrix4 transform,
     RenderBox child
   }) : super(child) {
     assert(transform != null);
     this.transform = transform;
   }
@@ skipping 43 matching lines @@
     sky.Point transformed = new sky.Point(transformed3.x, transformed3.y);
     super.hitTestChildren(result, position: transformed);
   }
 
   void paint(RenderObjectDisplayList canvas) {
     canvas.save();
     canvas.concat(_transform.storage);
     super.paint(canvas);
     canvas.restore();
   }
+
+  String debugDescribeSettings(String prefix) {
+    List<String> result = _transform.toString().split('\n').map((s) => '$prefix  $s\n').toList();
+    result.removeLast();
+    return '${super.debugDescribeSettings(prefix)}${prefix}transform matrix:\n${result.join()}';
+  }
 }
 
 typedef void SizeChangedCallback(sky.Size newSize);
 
 class RenderSizeObserver extends RenderProxyBox {
   RenderSizeObserver({
     this.callback,
     RenderBox child
   }) : super(child) {
     assert(callback != null);
@@ skipping 168 matching lines @@
 
   void defaultPaint(RenderObjectDisplayList canvas) {
     RenderBox child = firstChild;
     while (child != null) {
       assert(child.parentData is ParentDataType);
       canvas.paintChild(child, child.parentData.position);
       child = child.parentData.nextSibling;
     }
   }
 }