Replace setBoxDecoration() with a decoration property.

sky/sdk/lib/framework/layout2.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.
 
 library layout;
 
 // This version of layout.dart is an update to the other one, this one using new APIs.
 // It will not work in a stock Sky setup currently.
 
 import 'node.dart';
@@ skipping 486 matching lines @@
     hitTestChildren(result, x: x, y: y);
     result.add(this);
     return true;
   }
   void hitTestChildren(HitTestResult result, { double x, double y }) { }
 
   double width;
   double height;
 }
 
+// This must be immutable, because we won't notice when it changes
 class BoxDecoration {
   const BoxDecoration({
     this.backgroundColor
   });
 
   final int backgroundColor;
 }
 
 class RenderDecoratedBox extends RenderBox {
-  BoxDecoration _decoration;
 
   RenderDecoratedBox(BoxDecoration decoration) : _decoration = decoration;
 
-  void setBoxDecoration(BoxDecoration decoration) {
-    if (_decoration == decoration)
+  BoxDecoration _decoration;
+  BoxDecoration get decoration => _decoration;
+  void set decoration (BoxDecoration value) {
+    if (value == _decoration)
       return;
-    _decoration = decoration;
+    _decoration = value;
     markNeedsPaint();
   }
 
   void paint(RenderNodeDisplayList canvas) {
     assert(width != null);
     assert(height != null);
 
     if (_decoration == null)
       return;
 
     if (_decoration.backgroundColor != null) {
       sky.Paint paint = new sky.Paint()..color = _decoration.backgroundColor;
       canvas.drawRect(new sky.Rect()..setLTRB(0.0, 0.0, width, height), paint);
     }
   }
+

[abarth-chromium, 2015/05/27 16:29:39]

extraneous change?

 }
 
 
 // RENDER VIEW LAYOUT MANAGER
 
 class RenderView extends RenderNode {
 
   RenderView({
     RenderBox root,
     this.timeForRotation: const Duration(microseconds: 83333)
@@ skipping 124 matching lines @@
   // uses the maximum width provided by the parent
   // sizes itself to the height of its child stack
 
   RenderBlock({
     BoxDecoration decoration,
     EdgeDims padding: const EdgeDims(0.0, 0.0, 0.0, 0.0)
   }) : super(decoration), _padding = padding;
 
   EdgeDims _padding;
   EdgeDims get padding => _padding;
-  void set padding(EdgeDims value) {
+  void set padding (EdgeDims value) {

[abarth-chromium, 2015/05/27 16:29:40]

extraneous change?

     assert(value != null);
     if (_padding != value) {
       _padding = value;
       markNeedsLayout();
     }
   }
 
   void setParentData(RenderBox child) {
     if (child.parentData is! BlockParentData)
       child.parentData = new BlockParentData();
@@ skipping 262 matching lines @@
     canvas.paintChild(body, (body.parentData as BoxParentData).x, (body.parentData as BoxParentData).y);
     if (statusbar != null)
       canvas.paintChild(statusbar, (statusbar.parentData as BoxParentData).x, (statusbar.parentData as BoxParentData).y);
     if (toolbar != null)
       canvas.paintChild(toolbar, (toolbar.parentData as BoxParentData).x, (toolbar.parentData as BoxParentData).y);
     if (drawer != null)
       canvas.paintChild(drawer, (drawer.parentData as BoxParentData).x, (drawer.parentData as BoxParentData).y);
   }
 
 }