Borders.

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 'node.dart';
 import 'package:vector_math/vector_math.dart';
 import 'package:sky/framework/net/image_cache.dart' as image_cache;
@@ skipping 332 matching lines @@
       canvas.save();
       canvas.scale(widthScale, heightScale);
     }
     sky.Paint paint = new sky.Paint();
     canvas.drawImage(_image, 0.0, 0.0, paint);
     if (needsScale)
       canvas.restore();
   }
 }
 
+class BorderSide {
+  const BorderSide({
+    this.color: const sky.Color(0xFF000000),
+    this.width: 1.0
+  });
+  final sky.Color color;
+  final double width;
+
+  static const None = const BorderSide(width: 0.0);

[Matt Perry, 2015/06/04 16:58:33]

what's our naming scheme for class constants? I've been following the dart style
guide which recommends camelCase:
https://www.dartlang.org/articles/style-guide/#names

[Matt Perry, 2015/06/04 17:59:43]

Still curious what the decision was here. Should we:
1. Use CapsStyle naming.
2. Use camelCase naming.
3. Use whichever one we prefer?

+
+  int get hashCode {
+    int value = 373;
+    value = 37 * value * color.hashCode;
+    value = 37 * value * width.hashCode;
+    return value;
+  }
+  String toString() => 'BorderSide($color, $width)';
+}
+
+class Border {
+  const Border({
+    this.top: BorderSide.None,
+    this.right: BorderSide.None,
+    this.bottom: BorderSide.None,
+    this.left: BorderSide.None
+  });
+  const Border.all(BorderSide side) :
+    top = side,
+    right = side,
+    bottom = side,
+    left = side;
+  final BorderSide top;
+  final BorderSide right;
+  final BorderSide bottom;
+  final BorderSide left;
+
+  int get hashCode {
+    int value = 373;
+    value = 37 * value * top.hashCode;
+    value = 37 * value * right.hashCode;
+    value = 37 * value * bottom.hashCode;
+    value = 37 * value * left.hashCode;
+    return value;
+  }
+  String toString() => 'Border($top, $right, $bottom, $left)';
+}
+
 // This must be immutable, because we won't notice when it changes
 class BoxDecoration {
-  const BoxDecoration({this.backgroundColor});
+  const BoxDecoration({
+    this.backgroundColor,
+    this.border
+  });
 
   final sky.Color backgroundColor;
+  final Border border;
 }
 
 class RenderDecoratedBox extends RenderProxyBox {
 
   RenderDecoratedBox({
     BoxDecoration decoration,
     RenderBox child
-  }) : _decoration = decoration, super(child);
+  }) : _decoration = decoration, super(child) {
+    assert(_decoration != null);
+  }
 
   BoxDecoration _decoration;
   BoxDecoration get decoration => _decoration;
   void set decoration (BoxDecoration value) {
+    assert(value != null);
     if (value == _decoration)
       return;
     _decoration = value;
     markNeedsPaint();
   }
 
   void paint(RenderNodeDisplayList canvas) {
     assert(size.width != null);
     assert(size.height != null);
 
-    if (_decoration == null)
-      return;
-
     if (_decoration.backgroundColor != null) {
       sky.Paint paint = new sky.Paint()..color = _decoration.backgroundColor;
       canvas.drawRect(new sky.Rect.fromLTRB(0.0, 0.0, size.width, size.height), paint);
     }
+
+    if (_decoration.border != null) {
+      assert(_decoration.border.top != null);
+      assert(_decoration.border.right != null);
+      assert(_decoration.border.bottom != null);
+      assert(_decoration.border.left != null);
+
+      sky.Paint paint = new sky.Paint();
+      sky.Path path;
+
+      paint.color = _decoration.border.top.color;
+      path = new sky.Path();
+      path.moveTo(0.0,0.0);

[Matt Perry, 2015/06/04 16:58:33]

nit: space after ,

+      path.lineTo(_decoration.border.left.width, _decoration.border.top.width);
+      path.lineTo(size.width - _decoration.border.right.width, _decoration.border.top.width);
+      path.lineTo(size.width, 0.0);
+      path.close();
+      canvas.drawPath(path, paint);
+
+      paint.color = _decoration.border.right.color;
+      path = new sky.Path();
+      path.moveTo(size.width, 0.0);
+      path.lineTo(size.width - _decoration.border.right.width, _decoration.border.top.width);
+      path.lineTo(size.width - _decoration.border.right.width, size.height - _decoration.border.bottom.width);
+      path.lineTo(size.width, size.height);
+      path.close();
+      canvas.drawPath(path, paint);
+
+      paint.color = _decoration.border.bottom.color;
+      path = new sky.Path();
+      path.moveTo(size.width, size.height);
+      path.lineTo(size.width - _decoration.border.right.width, size.height - _decoration.border.bottom.width);
+      path.lineTo(_decoration.border.left.width, size.height - _decoration.border.bottom.width);
+      path.lineTo(0.0, size.height);
+      path.close();
+      canvas.drawPath(path, paint);
+
+      paint.color = _decoration.border.left.color;
+      path = new sky.Path();
+      path.moveTo(0.0, size.height);
+      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);
   }
 }
 
 class RenderTransform extends RenderProxyBox {
   RenderTransform({
     Matrix4 transform,
     RenderBox child
   }) : super(child) {
     assert(transform != null);
@@ skipping 161 matching lines @@
 
   void defaultPaint(RenderNodeDisplayList canvas) {
     RenderBox child = firstChild;
     while (child != null) {
       assert(child.parentData is ParentDataType);
       canvas.paintChild(child, child.parentData.position);
       child = child.parentData.nextSibling;
     }
   }
 }