Add even more careful checks around BoxConstraints (#3243)
I ran into a case where I was setting minHeight=∞ and then calling
layout() with that constraint, which is all kinds of bad. To try to
catch this earlier, this patch now provides a way to catch constraints
that are requiring infinite values.
We don't _always_ check this because there are valid uses for
BoxConstraints.biggest, e.g. as an additionalConstraint.
diff --git a/examples/layers/rendering/src/sector_layout.dart b/examples/layers/rendering/src/sector_layout.dart
index 069888c..3a9688c 100644
--- a/examples/layers/rendering/src/sector_layout.dart
+++ b/examples/layers/rendering/src/sector_layout.dart
@@ -43,7 +43,7 @@
bool get isNormalized => minDeltaRadius <= maxDeltaRadius && minDeltaTheta <= maxDeltaTheta;
@override
- bool get debugAssertIsNormalized {
+ bool debugAssertIsValid({ bool isAppliedConstraint: false }) {
assert(isNormalized);
return isNormalized;
}
diff --git a/packages/flutter/lib/src/rendering/block.dart b/packages/flutter/lib/src/rendering/block.dart
index 8ad17f4..1b8515b 100644
--- a/packages/flutter/lib/src/rendering/block.dart
+++ b/packages/flutter/lib/src/rendering/block.dart
@@ -171,7 +171,7 @@
@override
double getMinIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
switch (mainAxis) {
case Axis.horizontal:
return _getIntrinsicMainAxis(constraints, constraints.constrainWidth);
@@ -186,7 +186,7 @@
@override
double getMaxIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
switch (mainAxis) {
case Axis.horizontal:
return _getIntrinsicMainAxis(constraints, constraints.constrainWidth);
@@ -201,7 +201,7 @@
@override
double getMinIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
switch (mainAxis) {
case Axis.horizontal:
return _getIntrinsicCrossAxis(
@@ -216,7 +216,7 @@
@override
double getMaxIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
switch (mainAxis) {
case Axis.horizontal:
return _getIntrinsicCrossAxis(
diff --git a/packages/flutter/lib/src/rendering/box.dart b/packages/flutter/lib/src/rendering/box.dart
index 39ce2d6..0a4a03e 100644
--- a/packages/flutter/lib/src/rendering/box.dart
+++ b/packages/flutter/lib/src/rendering/box.dart
@@ -109,7 +109,7 @@
/// Returns new box constraints that are smaller by the given edge dimensions.
BoxConstraints deflate(EdgeInsets edges) {
assert(edges != null);
- assert(debugAssertIsNormalized);
+ assert(debugAssertIsValid());
double horizontal = edges.left + edges.right;
double vertical = edges.top + edges.bottom;
double deflatedMinWidth = math.max(0.0, minWidth - horizontal);
@@ -124,7 +124,7 @@
/// Returns new box constraints that remove the minimum width and height requirements.
BoxConstraints loosen() {
- assert(debugAssertIsNormalized);
+ assert(debugAssertIsValid());
return new BoxConstraints(
minWidth: 0.0,
maxWidth: maxWidth,
@@ -175,14 +175,14 @@
/// Returns the width that both satisfies the constraints and is as close as
/// possible to the given width.
double constrainWidth([double width = double.INFINITY]) {
- assert(debugAssertIsNormalized);
+ assert(debugAssertIsValid());
return width.clamp(minWidth, maxWidth);
}
/// Returns the height that both satisfies the constraints and is as close as
/// possible to the given height.
double constrainHeight([double height = double.INFINITY]) {
- assert(debugAssertIsNormalized);
+ assert(debugAssertIsValid());
return height.clamp(minHeight, maxHeight);
}
@@ -222,7 +222,7 @@
/// Whether the given size satisfies the constraints.
bool isSatisfiedBy(Size size) {
- assert(debugAssertIsNormalized);
+ assert(debugAssertIsValid());
return (minWidth <= size.width) && (size.width <= maxWidth) &&
(minHeight <= size.height) && (size.height <= maxHeight);
}
@@ -273,8 +273,8 @@
return b * t;
if (b == null)
return a * (1.0 - t);
- assert(a.debugAssertIsNormalized);
- assert(b.debugAssertIsNormalized);
+ assert(a.debugAssertIsValid());
+ assert(b.debugAssertIsValid());
return new BoxConstraints(
minWidth: ui.lerpDouble(a.minWidth, b.minWidth, t),
maxWidth: ui.lerpDouble(a.maxWidth, b.maxWidth, t),
@@ -303,7 +303,7 @@
}
@override
- bool get debugAssertIsNormalized {
+ bool debugAssertIsValid({ bool isAppliedConstraint: false }) {
assert(() {
if (minWidth.isNaN || maxWidth.isNaN || minHeight.isNaN || maxHeight.isNaN) {
List<String> affectedFieldsList = <String>[];
@@ -340,7 +340,16 @@
throw new FlutterError('BoxConstraints has non-normalized width constraints.\n$this');
if (maxHeight < minHeight)
throw new FlutterError('BoxConstraints has non-normalized height constraints.\n$this');
- return isNormalized;
+ if (isAppliedConstraint) {
+ if (minWidth.isInfinite && minHeight.isInfinite)
+ throw new FlutterError('BoxConstraints requires infinite width and infinite height.\n$this');
+ if (minWidth.isInfinite)
+ throw new FlutterError('BoxConstraints requires infinite width.\n$this');
+ if (minHeight.isInfinite)
+ throw new FlutterError('BoxConstraints requires infinite height.\n$this');
+ }
+ assert(isNormalized);
+ return true;
});
return isNormalized;
}
@@ -356,13 +365,13 @@
@override
bool operator ==(dynamic other) {
- assert(debugAssertIsNormalized);
+ assert(debugAssertIsValid());
if (identical(this, other))
return true;
if (other is! BoxConstraints)
return false;
final BoxConstraints typedOther = other;
- assert(typedOther.debugAssertIsNormalized);
+ assert(typedOther.debugAssertIsValid());
return minWidth == typedOther.minWidth &&
maxWidth == typedOther.maxWidth &&
minHeight == typedOther.minHeight &&
@@ -371,7 +380,7 @@
@override
int get hashCode {
- assert(debugAssertIsNormalized);
+ assert(debugAssertIsValid());
return hashValues(minWidth, maxWidth, minHeight, maxHeight);
}
@@ -451,7 +460,7 @@
///
/// Override in subclasses that implement [performLayout].
double getMinIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return constraints.constrainWidth(0.0);
}
@@ -460,7 +469,7 @@
///
/// Override in subclasses that implement [performLayout].
double getMaxIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return constraints.constrainWidth(0.0);
}
@@ -469,7 +478,7 @@
///
/// Override in subclasses that implement [performLayout].
double getMinIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return constraints.constrainHeight(0.0);
}
@@ -482,7 +491,7 @@
///
/// Override in subclasses that implement [performLayout].
double getMaxIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return constraints.constrainHeight(0.0);
}
diff --git a/packages/flutter/lib/src/rendering/custom_layout.dart b/packages/flutter/lib/src/rendering/custom_layout.dart
index 57b73c2..365411f 100644
--- a/packages/flutter/lib/src/rendering/custom_layout.dart
+++ b/packages/flutter/lib/src/rendering/custom_layout.dart
@@ -57,7 +57,7 @@
);
}
try {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid(isAppliedConstraint: true));
} on AssertionError catch (exception) {
throw new FlutterError(
'The $this custom multichild layout delegate provided invalid box constraints for the child with id "$childId".\n'
@@ -232,7 +232,7 @@
}
Size _getSize(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return constraints.constrain(_delegate.getSize(constraints));
}
diff --git a/packages/flutter/lib/src/rendering/editable_line.dart b/packages/flutter/lib/src/rendering/editable_line.dart
index 013e7e9..c987ca5 100644
--- a/packages/flutter/lib/src/rendering/editable_line.dart
+++ b/packages/flutter/lib/src/rendering/editable_line.dart
@@ -133,25 +133,25 @@
@override
double getMinIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return constraints.constrainWidth(0.0);
}
@override
double getMaxIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return constraints.constrainWidth(0.0);
}
@override
double getMinIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return constraints.constrainHeight(_preferredHeight);
}
@override
double getMaxIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return constraints.constrainHeight(_preferredHeight);
}
@@ -190,7 +190,7 @@
// TODO(abarth): This logic should live in TextPainter and be shared with RenderParagraph.
void _layoutText(BoxConstraints constraints) {
assert(constraints != null);
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (_constraintsForCurrentLayout == constraints)
return; // already cached this layout
_textPainter.maxWidth = constraints.maxWidth;
diff --git a/packages/flutter/lib/src/rendering/flex.dart b/packages/flutter/lib/src/rendering/flex.dart
index 518af43..c9060fd 100644
--- a/packages/flutter/lib/src/rendering/flex.dart
+++ b/packages/flutter/lib/src/rendering/flex.dart
@@ -144,7 +144,7 @@
double _getIntrinsicSize({ BoxConstraints constraints,
FlexDirection sizingDirection,
_ChildSizingFunction childSize }) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
// http://www.w3.org/TR/2015/WD-css-flexbox-1-20150514/#intrinsic-sizes
if (_direction == sizingDirection) {
// INTRINSIC MAIN SIZE
diff --git a/packages/flutter/lib/src/rendering/grid.dart b/packages/flutter/lib/src/rendering/grid.dart
index b3f53b5..50974e9 100644
--- a/packages/flutter/lib/src/rendering/grid.dart
+++ b/packages/flutter/lib/src/rendering/grid.dart
@@ -430,25 +430,25 @@
@override
double getMinIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return _delegate.getMinIntrinsicWidth(constraints, virtualChildCount);
}
@override
double getMaxIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return _delegate.getMaxIntrinsicWidth(constraints, virtualChildCount);
}
@override
double getMinIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return _delegate.getMinIntrinsicHeight(constraints, virtualChildCount);
}
@override
double getMaxIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return _delegate.getMaxIntrinsicHeight(constraints, virtualChildCount);
}
diff --git a/packages/flutter/lib/src/rendering/image.dart b/packages/flutter/lib/src/rendering/image.dart
index 91e2ece..029a79b 100644
--- a/packages/flutter/lib/src/rendering/image.dart
+++ b/packages/flutter/lib/src/rendering/image.dart
@@ -205,7 +205,7 @@
@override
double getMinIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (_width == null && _height == null)
return constraints.constrainWidth(0.0);
return _sizeForConstraints(constraints).width;
@@ -213,13 +213,13 @@
@override
double getMaxIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return _sizeForConstraints(constraints).width;
}
@override
double getMinIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (_width == null && _height == null)
return constraints.constrainHeight(0.0);
return _sizeForConstraints(constraints).height;
@@ -227,7 +227,7 @@
@override
double getMaxIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return _sizeForConstraints(constraints).height;
}
diff --git a/packages/flutter/lib/src/rendering/list.dart b/packages/flutter/lib/src/rendering/list.dart
index abea020..ded1f43 100644
--- a/packages/flutter/lib/src/rendering/list.dart
+++ b/packages/flutter/lib/src/rendering/list.dart
@@ -82,7 +82,7 @@
}
double _getIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
switch (mainAxis) {
case Axis.vertical:
return constraints.constrainWidth(0.0);
@@ -102,7 +102,7 @@
}
double _getIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
switch (mainAxis) {
case Axis.vertical:
return constraints.constrainHeight(_preferredExtent);
diff --git a/packages/flutter/lib/src/rendering/object.dart b/packages/flutter/lib/src/rendering/object.dart
index e4f2112..154426d 100644
--- a/packages/flutter/lib/src/rendering/object.dart
+++ b/packages/flutter/lib/src/rendering/object.dart
@@ -382,9 +382,25 @@
/// Whether the constraint is expressed in a consistent manner.
bool get isNormalized;
- /// Same as [isNormalized] but, in checked mode, throws an exception
- /// if isNormalized is false.
- bool get debugAssertIsNormalized;
+ /// Asserts that the constraints are valid.
+ ///
+ /// This might involve checks more detailed than [isNormalized].
+ ///
+ /// For example, the [BoxConstraints] subclass verifies that the
+ /// constraints are not [NaN].
+ ///
+ /// If the [isAppliedConstraint] argument is true, then even
+ /// stricter rules are enforced. This argument is set to true when
+ /// checking constraints that are about to be applied to a
+ /// [RenderObject] during layout, as opposed to constraints that may
+ /// be further affected by other constraints. For example, the
+ /// asserts for verifying the validity of
+ /// [RenderConstrainedBox.additionalConstraints] do not set this
+ /// argument, but the asserts for verifying the argument passed to
+ /// the [layout] method do.
+ ///
+ /// Returns the same as [isNormalized] if asserts are disabled.
+ bool debugAssertIsValid({ bool isAppliedConstraint: false });
}
typedef void RenderObjectVisitor(RenderObject child);
@@ -1136,7 +1152,7 @@
/// work to update its layout information.
void layout(Constraints constraints, { bool parentUsesSize: false }) {
assert(constraints != null);
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid(isAppliedConstraint: true));
assert(!_debugDoingThisResize);
assert(!_debugDoingThisLayout);
final RenderObject parent = this.parent;
diff --git a/packages/flutter/lib/src/rendering/paragraph.dart b/packages/flutter/lib/src/rendering/paragraph.dart
index d6e730a..821a7ce 100644
--- a/packages/flutter/lib/src/rendering/paragraph.dart
+++ b/packages/flutter/lib/src/rendering/paragraph.dart
@@ -36,7 +36,7 @@
// TODO(abarth): This logic should live in TextPainter and be shared with RenderEditableLine.
void _layoutText(BoxConstraints constraints) {
assert(constraints != null);
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (_constraintsForCurrentLayout == constraints)
return; // already cached this layout
_textPainter.maxWidth = constraints.maxWidth;
@@ -71,13 +71,13 @@
@override
double getMinIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return _getIntrinsicHeight(constraints);
}
@override
double getMaxIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return _getIntrinsicHeight(constraints);
}
diff --git a/packages/flutter/lib/src/rendering/proxy_box.dart b/packages/flutter/lib/src/rendering/proxy_box.dart
index efb9998..b42b80f 100644
--- a/packages/flutter/lib/src/rendering/proxy_box.dart
+++ b/packages/flutter/lib/src/rendering/proxy_box.dart
@@ -37,7 +37,7 @@
@override
double getMinIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child != null)
return child.getMinIntrinsicWidth(constraints);
return super.getMinIntrinsicWidth(constraints);
@@ -45,7 +45,7 @@
@override
double getMaxIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child != null)
return child.getMaxIntrinsicWidth(constraints);
return super.getMaxIntrinsicWidth(constraints);
@@ -53,7 +53,7 @@
@override
double getMinIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child != null)
return child.getMinIntrinsicHeight(constraints);
return super.getMinIntrinsicHeight(constraints);
@@ -61,7 +61,7 @@
@override
double getMaxIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child != null)
return child.getMaxIntrinsicHeight(constraints);
return super.getMaxIntrinsicHeight(constraints);
@@ -170,7 +170,7 @@
BoxConstraints additionalConstraints
}) : _additionalConstraints = additionalConstraints, super(child) {
assert(additionalConstraints != null);
- assert(additionalConstraints.debugAssertIsNormalized);
+ assert(additionalConstraints.debugAssertIsValid());
}
/// Additional constraints to apply to [child] during layout
@@ -178,7 +178,7 @@
BoxConstraints _additionalConstraints;
void set additionalConstraints (BoxConstraints newConstraints) {
assert(newConstraints != null);
- assert(newConstraints.debugAssertIsNormalized);
+ assert(newConstraints.debugAssertIsValid());
if (_additionalConstraints == newConstraints)
return;
_additionalConstraints = newConstraints;
@@ -187,7 +187,7 @@
@override
double getMinIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child != null)
return child.getMinIntrinsicWidth(_additionalConstraints.enforce(constraints));
return _additionalConstraints.enforce(constraints).constrainWidth(0.0);
@@ -195,7 +195,7 @@
@override
double getMaxIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child != null)
return child.getMaxIntrinsicWidth(_additionalConstraints.enforce(constraints));
return _additionalConstraints.enforce(constraints).constrainWidth(0.0);
@@ -203,7 +203,7 @@
@override
double getMinIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child != null)
return child.getMinIntrinsicHeight(_additionalConstraints.enforce(constraints));
return _additionalConstraints.enforce(constraints).constrainHeight(0.0);
@@ -211,7 +211,7 @@
@override
double getMaxIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child != null)
return child.getMaxIntrinsicHeight(_additionalConstraints.enforce(constraints));
return _additionalConstraints.enforce(constraints).constrainHeight(0.0);
@@ -317,7 +317,7 @@
}
Size _applyAspectRatio(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
assert(() {
if (!constraints.hasBoundedWidth && !constraints.hasBoundedHeight) {
throw new FlutterError(
@@ -446,13 +446,13 @@
@override
double getMinIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return getMaxIntrinsicWidth(constraints);
}
@override
double getMaxIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child == null)
return constraints.constrainWidth(0.0);
double childResult = child.getMaxIntrinsicWidth(constraints);
@@ -461,7 +461,7 @@
@override
double getMinIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child == null)
return constraints.constrainHeight(0.0);
double childResult = child.getMinIntrinsicHeight(_getInnerConstraints(constraints));
@@ -470,7 +470,7 @@
@override
double getMaxIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child == null)
return constraints.constrainHeight(0.0);
double childResult = child.getMaxIntrinsicHeight(_getInnerConstraints(constraints));
@@ -522,7 +522,7 @@
@override
double getMinIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child == null)
return constraints.constrainWidth(0.0);
return child.getMinIntrinsicWidth(_getInnerConstraints(constraints));
@@ -530,7 +530,7 @@
@override
double getMaxIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child == null)
return constraints.constrainWidth(0.0);
return child.getMaxIntrinsicWidth(_getInnerConstraints(constraints));
@@ -538,13 +538,13 @@
@override
double getMinIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return getMaxIntrinsicHeight(constraints);
}
@override
double getMaxIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child == null)
return constraints.constrainHeight(0.0);
return child.getMaxIntrinsicHeight(constraints);
diff --git a/packages/flutter/lib/src/rendering/rotated_box.dart b/packages/flutter/lib/src/rendering/rotated_box.dart
index 059362f..9f5ffe5 100644
--- a/packages/flutter/lib/src/rendering/rotated_box.dart
+++ b/packages/flutter/lib/src/rendering/rotated_box.dart
@@ -41,7 +41,7 @@
@override
double getMinIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child != null)
return _isVertical ? child.getMinIntrinsicHeight(constraints.flipped) : child.getMinIntrinsicWidth(constraints);
return super.getMinIntrinsicWidth(constraints);
@@ -49,7 +49,7 @@
@override
double getMaxIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child != null)
return _isVertical ? child.getMaxIntrinsicHeight(constraints.flipped) : child.getMaxIntrinsicWidth(constraints);
return super.getMaxIntrinsicWidth(constraints);
@@ -57,7 +57,7 @@
@override
double getMinIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child != null)
return _isVertical ? child.getMinIntrinsicWidth(constraints.flipped) : child.getMinIntrinsicHeight(constraints);
return super.getMinIntrinsicHeight(constraints);
@@ -65,7 +65,7 @@
@override
double getMaxIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child != null)
return _isVertical ? child.getMaxIntrinsicWidth(constraints.flipped) : child.getMaxIntrinsicHeight(constraints);
return super.getMaxIntrinsicHeight(constraints);
diff --git a/packages/flutter/lib/src/rendering/shifted_box.dart b/packages/flutter/lib/src/rendering/shifted_box.dart
index 972b02d..83da423 100644
--- a/packages/flutter/lib/src/rendering/shifted_box.dart
+++ b/packages/flutter/lib/src/rendering/shifted_box.dart
@@ -17,7 +17,7 @@
@override
double getMinIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child != null)
return child.getMinIntrinsicWidth(constraints);
return super.getMinIntrinsicWidth(constraints);
@@ -25,7 +25,7 @@
@override
double getMaxIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child != null)
return child.getMaxIntrinsicWidth(constraints);
return super.getMaxIntrinsicWidth(constraints);
@@ -33,7 +33,7 @@
@override
double getMinIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child != null)
return child.getMinIntrinsicHeight(constraints);
return super.getMinIntrinsicHeight(constraints);
@@ -41,7 +41,7 @@
@override
double getMaxIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child != null)
return child.getMaxIntrinsicHeight(constraints);
return super.getMaxIntrinsicHeight(constraints);
@@ -112,7 +112,7 @@
@override
double getMinIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
double totalPadding = padding.left + padding.right;
if (child != null)
return constraints.constrainWidth(child.getMinIntrinsicWidth(constraints.deflate(padding)) + totalPadding);
@@ -121,7 +121,7 @@
@override
double getMaxIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
double totalPadding = padding.left + padding.right;
if (child != null)
return constraints.constrainWidth(child.getMaxIntrinsicWidth(constraints.deflate(padding)) + totalPadding);
@@ -130,7 +130,7 @@
@override
double getMinIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
double totalPadding = padding.top + padding.bottom;
if (child != null)
return constraints.constrainHeight(child.getMinIntrinsicHeight(constraints.deflate(padding)) + totalPadding);
@@ -139,7 +139,7 @@
@override
double getMaxIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
double totalPadding = padding.top + padding.bottom;
if (child != null)
return constraints.constrainHeight(child.getMaxIntrinsicHeight(constraints.deflate(padding)) + totalPadding);
@@ -494,25 +494,25 @@
@override
double getMinIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return constraints.minWidth;
}
@override
double getMaxIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return constraints.minWidth;
}
@override
double getMinIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return constraints.minHeight;
}
@override
double getMaxIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return constraints.minHeight;
}
@@ -567,25 +567,25 @@
@override
double getMinIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return constraints.constrainWidth(_requestedSize.width);
}
@override
double getMaxIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return constraints.constrainWidth(_requestedSize.width);
}
@override
double getMinIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return constraints.constrainHeight(_requestedSize.height);
}
@override
double getMaxIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return constraints.constrainHeight(_requestedSize.height);
}
@@ -683,7 +683,7 @@
@override
double getMinIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child != null)
return constraints.constrainWidth(child.getMinIntrinsicWidth(_getInnerConstraints(constraints)));
return constraints.constrainWidth(_getInnerConstraints(constraints).constrainWidth(0.0));
@@ -691,7 +691,7 @@
@override
double getMaxIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child != null)
return constraints.constrainWidth(child.getMaxIntrinsicWidth(_getInnerConstraints(constraints)));
return constraints.constrainWidth(_getInnerConstraints(constraints).constrainWidth(0.0));
@@ -699,7 +699,7 @@
@override
double getMinIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child != null)
return constraints.constrainHeight(child.getMinIntrinsicHeight(_getInnerConstraints(constraints)));
return constraints.constrainHeight(_getInnerConstraints(constraints).constrainHeight(0.0));
@@ -707,7 +707,7 @@
@override
double getMaxIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child != null)
return constraints.constrainHeight(child.getMaxIntrinsicHeight(_getInnerConstraints(constraints)));
return constraints.constrainHeight(_getInnerConstraints(constraints).constrainHeight(0.0));
@@ -779,25 +779,25 @@
@override
double getMinIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return _getSize(constraints).width;
}
@override
double getMaxIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return _getSize(constraints).width;
}
@override
double getMinIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return _getSize(constraints).height;
}
@override
double getMaxIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return _getSize(constraints).height;
}
@@ -813,7 +813,7 @@
void performLayout() {
if (child != null) {
BoxConstraints childConstraints = delegate.getConstraintsForChild(constraints);
- assert(childConstraints.debugAssertIsNormalized);
+ assert(childConstraints.debugAssertIsValid(isAppliedConstraint: true));
child.layout(childConstraints, parentUsesSize: !childConstraints.isTight);
final BoxParentData childParentData = child.parentData;
childParentData.offset = delegate.getPositionForChild(size, childConstraints.isTight ? childConstraints.smallest : child.size);
diff --git a/packages/flutter/lib/src/rendering/stack.dart b/packages/flutter/lib/src/rendering/stack.dart
index 5a865db..23692a5 100644
--- a/packages/flutter/lib/src/rendering/stack.dart
+++ b/packages/flutter/lib/src/rendering/stack.dart
@@ -225,7 +225,7 @@
@override
double getMinIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
double width = constraints.minWidth;
RenderBox child = firstChild;
while (child != null) {
@@ -241,7 +241,7 @@
@override
double getMaxIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
bool hasNonPositionedChildren = false;
double width = constraints.minWidth;
RenderBox child = firstChild;
@@ -262,7 +262,7 @@
@override
double getMinIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
double height = constraints.minHeight;
RenderBox child = firstChild;
while (child != null) {
@@ -278,7 +278,7 @@
@override
double getMaxIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
bool hasNonPositionedChildren = false;
double height = constraints.minHeight;
RenderBox child = firstChild;
diff --git a/packages/flutter/lib/src/rendering/table.dart b/packages/flutter/lib/src/rendering/table.dart
index cb2a586..3e6dda3 100644
--- a/packages/flutter/lib/src/rendering/table.dart
+++ b/packages/flutter/lib/src/rendering/table.dart
@@ -636,7 +636,7 @@
@override
double getMinIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
assert(_children.length == rows * columns);
double totalMinWidth = 0.0;
for (int x = 0; x < columns; x += 1) {
@@ -649,7 +649,7 @@
@override
double getMaxIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
assert(_children.length == rows * columns);
double totalMaxWidth = 0.0;
for (int x = 0; x < columns; x += 1) {
@@ -664,7 +664,7 @@
double getMinIntrinsicHeight(BoxConstraints constraints) {
// winner of the 2016 world's most expensive intrinsic dimension function award
// honorable mention, most likely to improve if taught about memoization award
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
assert(_children.length == rows * columns);
final List<double> widths = computeColumnWidths(constraints);
double rowTop = 0.0;
diff --git a/packages/flutter/lib/src/rendering/viewport.dart b/packages/flutter/lib/src/rendering/viewport.dart
index a260c52..08f8726 100644
--- a/packages/flutter/lib/src/rendering/viewport.dart
+++ b/packages/flutter/lib/src/rendering/viewport.dart
@@ -246,7 +246,7 @@
@override
double getMinIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child != null)
return constraints.constrainWidth(child.getMinIntrinsicWidth(_getInnerConstraints(constraints)));
return super.getMinIntrinsicWidth(constraints);
@@ -254,7 +254,7 @@
@override
double getMaxIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child != null)
return constraints.constrainWidth(child.getMaxIntrinsicWidth(_getInnerConstraints(constraints)));
return super.getMaxIntrinsicWidth(constraints);
@@ -262,7 +262,7 @@
@override
double getMinIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child != null)
return constraints.constrainHeight(child.getMinIntrinsicHeight(_getInnerConstraints(constraints)));
return super.getMinIntrinsicHeight(constraints);
@@ -270,7 +270,7 @@
@override
double getMaxIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
if (child != null)
return constraints.constrainHeight(child.getMaxIntrinsicHeight(_getInnerConstraints(constraints)));
return super.getMaxIntrinsicHeight(constraints);
diff --git a/packages/flutter/lib/src/widgets/lazy_block.dart b/packages/flutter/lib/src/widgets/lazy_block.dart
index f142510..ffec2b3 100644
--- a/packages/flutter/lib/src/widgets/lazy_block.dart
+++ b/packages/flutter/lib/src/widgets/lazy_block.dart
@@ -264,13 +264,13 @@
@override
double getMinIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return getIntrinsicWidth(constraints);
}
@override
double getMaxIntrinsicWidth(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return getIntrinsicWidth(constraints);
}
@@ -286,13 +286,13 @@
@override
double getMinIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return getIntrinsicHeight(constraints);
}
@override
double getMaxIntrinsicHeight(BoxConstraints constraints) {
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
return getIntrinsicHeight(constraints);
}
diff --git a/packages/flutter/test/rendering/constraints_test.dart b/packages/flutter/test/rendering/constraints_test.dart
index 98d8d9b..f8fb527 100644
--- a/packages/flutter/test/rendering/constraints_test.dart
+++ b/packages/flutter/test/rendering/constraints_test.dart
@@ -37,7 +37,7 @@
result = 'no exception';
try {
BoxConstraints constraints = new BoxConstraints(minWidth: double.NAN, maxWidth: double.NAN, minHeight: 2.0, maxHeight: double.NAN);
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
} on FlutterError catch (e) {
result = '$e';
}
@@ -49,7 +49,7 @@
result = 'no exception';
try {
BoxConstraints constraints = new BoxConstraints(minHeight: double.NAN);
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
} on FlutterError catch (e) {
result = '$e';
}
@@ -61,7 +61,7 @@
result = 'no exception';
try {
BoxConstraints constraints = new BoxConstraints(minHeight: double.NAN, maxWidth: 0.0/0.0);
- assert(constraints.debugAssertIsNormalized);
+ assert(constraints.debugAssertIsValid());
} on FlutterError catch (e) {
result = '$e';
}