BlockViewport didn't implement the intrinsic dimension functions correctly.

sky/sdk/lib/rendering/block.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 'box.dart';
 import 'object.dart';
 
 class BlockParentData extends BoxParentData with ContainerParentDataMixin<RenderBox> { }
 
 abstract class RenderBlockBase extends RenderBox with ContainerRenderObjectMixin<RenderBox, BlockParentData>,
                                                       RenderBoxContainerDefaultsMixin<RenderBox, BlockParentData> {
 
   // lays out RenderBox children in a vertical stack
   // uses the maximum width provided by the parent
 
   RenderBlockBase({
     List<RenderBox> children
   }) {
     addAll(children);
   }
 
   void setupParentData(RenderBox child) {
     if (child.parentData is! BlockParentData)
       child.parentData = new BlockParentData();
   }
 
+  double computeDistanceToActualBaseline(TextBaseline baseline) {
+    return defaultComputeDistanceToFirstActualBaseline(baseline);
+  }
+
+  double _childrenHeight;
+  double get childrenHeight => _childrenHeight;
+
+  void markNeedsLayout() {
+    _childrenHeight = null;
+    super.markNeedsLayout();
+  }
+
+  void performLayout() {
+    assert(constraints is BoxConstraints);
+    double width = constraints.constrainWidth(constraints.maxWidth);
+    BoxConstraints innerConstraints = _getInnerConstraintsForWidth(width);
+    double y = 0.0;
+    RenderBox child = firstChild;
+    while (child != null) {
+      child.layout(innerConstraints, parentUsesSize: true);
+      assert(child.parentData is BlockParentData);
+      child.parentData.position = new Point(0.0, y);
+      y += child.size.height;
+      child = child.parentData.nextSibling;
+    }
+    _childrenHeight = y;
+  }
+
+  void hitTestChildren(HitTestResult result, { Point position }) {
+    defaultHitTestChildren(result, position: position);
+  }
+
+  void paint(PaintingCanvas canvas, Offset offset) {
+    defaultPaint(canvas, offset);
+  }
+
+}
+
+class RenderBlock extends RenderBlockBase {
+
+  // sizes itself to the height of its child stack
+
+  RenderBlock({ List<RenderBox> children }) : super(children: children);
+
   double getMinIntrinsicWidth(BoxConstraints constraints) {
     double width = 0.0;
     BoxConstraints innerConstraints = constraints.widthConstraints();
     RenderBox child = firstChild;
     while (child != null) {
       width = math.max(width, child.getMinIntrinsicWidth(innerConstraints));
       assert(child.parentData is BlockParentData);
       child = child.parentData.nextSibling;
     }
     return width;
@@ skipping 31 matching lines @@
   }
 
   double getMinIntrinsicHeight(BoxConstraints constraints) {
     return _getIntrinsicHeight(constraints);
   }
 
   double getMaxIntrinsicHeight(BoxConstraints constraints) {
     return _getIntrinsicHeight(constraints);
   }
 
-  double computeDistanceToActualBaseline(TextBaseline baseline) {
-    return defaultComputeDistanceToFirstActualBaseline(baseline);
-  }
-
-  double _childrenHeight;
-  double get childrenHeight => _childrenHeight;
-
-  void markNeedsLayout() {
-    _childrenHeight = null;
-    super.markNeedsLayout();
-  }
-
-  void performLayout() {
-    assert(constraints is BoxConstraints);
-    double width = constraints.constrainWidth(constraints.maxWidth);
-    BoxConstraints innerConstraints = _getInnerConstraintsForWidth(width);
-    double y = 0.0;
-    RenderBox child = firstChild;
-    while (child != null) {
-      child.layout(innerConstraints, parentUsesSize: true);
-      assert(child.parentData is BlockParentData);
-      child.parentData.position = new Point(0.0, y);
-      y += child.size.height;
-      child = child.parentData.nextSibling;
-    }
-    _childrenHeight = y;
-  }
-
-  void hitTestChildren(HitTestResult result, { Point position }) {
-    defaultHitTestChildren(result, position: position);
-  }
-
-  void paint(PaintingCanvas canvas, Offset offset) {
-    defaultPaint(canvas, offset);
-  }
-
-}
-
-class RenderBlock extends RenderBlockBase {
-
-  // sizes itself to the height of its child stack
-
-  RenderBlock({ List<RenderBox> children }) : super(children: children);
-
   bool _hasVisualOverflow = false;
 
   void performLayout() {
     super.performLayout();
     size = constraints.constrain(new Size(constraints.maxWidth, childrenHeight));
     assert(!size.isInfinite);
 
     // FIXME(eseidel): Block lays out its children with unconstrained height
     // yet itself remains constrained. Remember that our children wanted to
     // be taller than we are so we know to clip them (and not cause confusing
@@ skipping 41 matching lines @@
   double _startOffset;
   double get startOffset => _startOffset;
   void set startOffset(double value) {
     if (value == _startOffset)
       return;
     _startOffset = value;
     if (!_inCallback)
       markNeedsLayout();
   }
 
+  double getMinIntrinsicWidth(BoxConstraints constraints) {
+    return constraints.constrainWidth();
+  }
+
+  double getMaxIntrinsicWidth(BoxConstraints constraints) {
+    return constraints.constrainWidth();
+  }
+
+  double getMinIntrinsicHeight(BoxConstraints constraints) {
+    return constraints.constrainHeight();
+  }
+
+  double getMaxIntrinsicHeight(BoxConstraints constraints) {
+    return constraints.constrainHeight();
+  }
+
   bool get sizedByParent => true;
 
   void performResize() {
     size = constraints.biggest;
     assert(!size.isInfinite);
   }
 
   bool get debugDoesLayoutWithCallback => true;
   void performLayout() {
     if (_callback != null) {
       try {
         _inCallback = true;
         invokeLayoutCallback(_callback);
       } finally {
         _inCallback = false;
       }
     }
     super.performLayout();
   }
 
   void paint(PaintingCanvas canvas, Offset offset) {
     canvas.save();
     canvas.clipRect(offset & size);
     super.paint(canvas, offset.translate(0.0, _startOffset));
     canvas.restore();
   }
 
 }