Implement RenderImage and Image for Sky

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 'node.dart';
 import 'dart:sky' as sky;
+import 'package:sky/framework/net/image_cache.dart' as image_cache;
 
 // GENERIC BOX RENDERING
 // Anything that has a concept of x, y, width, height is going to derive from this
 
 class EdgeDims {
   // used for e.g. padding
   const EdgeDims(this.top, this.right, this.bottom, this.left);
   const EdgeDims.all(double value)
       : top = value, right = value, bottom = value, left = value;
 
@@ skipping 219 matching lines @@
       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));
       }
     }
   }
 
 }
 
+class RenderImage extends RenderBox {
+
+  RenderImage(String src, sky.Size dimensions) {
+    requestedSize = dimensions;
+    image_cache.load(src, (result) {
+      _image = result;
+      markNeedsLayout();
+      markNeedsPaint();
+    });
+  }
+
+  sky.Image _image;
+  sky.Size _requestedSize;
+  sky.Size get requestedSize => _requestedSize;
+  void set requestedSize (sky.Size value) {
+    if (value == _requestedSize)
+      return;
+    _requestedSize = value;
+    markNeedsLayout();
+    markNeedsPaint();

[abarth-chromium, 2015/06/03 17:53:21]

No need to call markNeedsPaint here.  markNeedsLayout will trigger a
markNeedsPaint automatically if something about the size ends up mattering.

[jackson, 2015/06/03 22:06:26]

Done.

+  }
+
+  void _updateSizeFromImage() {

[abarth-chromium, 2015/06/03 17:53:21]

This function has one caller, performLayout.  We should just inline it into that
one caller.

[jackson, 2015/06/03 22:06:26]

Done.

+    // If both height and width are specified, we don't care about image dimensions
+    if (requestedSize.width != null && requestedSize.height != null)
+      size = constraints.constrain(requestedSize);

[abarth-chromium, 2015/06/03 17:53:21]

Should we return in this case?

[jackson, 2015/06/03 22:06:26]

Done.

+
+    // If there's no image, we can't size ourselves
+    if (_image == null)
+      return;
+
+    // If neither height nor width are specified, use inherent image dimensions
+    // If only one dimension is specified, adjust the other dimension to
+    // maintain the aspect ratio
+    if (requestedSize.width == null) {
+      if (requestedSize.height == null) {
+        size = constraints.constrain(new sky.Size(_image.width, _image.height));
+      } else {
+        num width = requestedSize.height * _image.width / _image.height;

[abarth-chromium, 2015/06/03 17:53:21]

s/num/double/  It's always a double 

[jackson, 2015/06/03 22:06:26]

Done.

+        size = constraints.constrain(new sky.Size(width, requestedSize.height));
+      }
+    } else if (requestedSize.height == null) {
+      num height = requestedSize.width * _image.height / _image.width;
+      size = constraints.constrain(new sky.Size(requestedSize.width, height));
+    } else {
+      size = constraints.constrain(requestedSize);
+    }
+  }
+
+  void performLayout() {
+    _updateSizeFromImage();
+  }
+
+  void paint(RenderNodeDisplayList canvas) {
+    if (_image == null) return;
+    double widthScale = size.width / _image.width;
+    double heightScale = size.height / _image.height;
+    canvas.save();
+    canvas.scale(widthScale, heightScale);

[abarth-chromium, 2015/06/03 17:53:21]

We should skip the save/scale/restore dance if size.width == _image.width &&
size.height == _image.height.  Calling save/scale/restore is expensive and
unnecessary in the common case that the size of the RenderImage matches the
intrinsic dimensions of the underlying image object.

[jackson, 2015/06/03 22:06:26]

Done.

+    sky.Paint paint = new sky.Paint();
+    canvas.drawImage(_image, 0.0, 0.0, paint);
+    canvas.restore();
+  }
+}
+
 // This must be immutable, because we won't notice when it changes
 class BoxDecoration {
   // TODO(mpcomplete): go through and change the users of this class to pass
   // a Color object.
   BoxDecoration({
     backgroundColor
   }) : backgroundColor = new sky.Color(backgroundColor);
 
   final sky.Color backgroundColor;
 }
@@ skipping 131 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;
     }
   }
 }