Don't push lines when height is unconstrained, even if current height is bigger than NS_UNCONSTRAINEDSIZE. b=265027 r+sr=roc a=mkaply

layout/generic/nsBlockFrame.cpp

@@ -4178,8 +4178,10 @@ nsBlockFrame::PlaceLine(nsBlockReflowState& aState,
   // fits; we'll assume it does, so that the maximum width will get
   // updated below. The line will be reflowed again and pushed then
   // if necessary.
-  if ((mLines.front() != aLine) && (newY > aState.mBottomEdge)
-      && !aUpdateMaximumWidth) {
+  if (mLines.front() != aLine &&
+      newY > aState.mBottomEdge &&
+      aState.mBottomEdge != NS_UNCONSTRAINEDSIZE &&
+      !aUpdateMaximumWidth) {
     // Push this line and all of it's children and anything else that
     // follows to our next-in-flow
     NS_ASSERTION((aState.mCurrentLine == aLine), "oops");

layout/html/base/src/nsBlockFrame.cpp

@@ -4178,8 +4178,10 @@ nsBlockFrame::PlaceLine(nsBlockReflowState& aState,
   // fits; we'll assume it does, so that the maximum width will get
   // updated below. The line will be reflowed again and pushed then
   // if necessary.
-  if ((mLines.front() != aLine) && (newY > aState.mBottomEdge)
-      && !aUpdateMaximumWidth) {
+  if (mLines.front() != aLine &&
+      newY > aState.mBottomEdge &&
+      aState.mBottomEdge != NS_UNCONSTRAINEDSIZE &&
+      !aUpdateMaximumWidth) {
     // Push this line and all of it's children and anything else that
     // follows to our next-in-flow
     NS_ASSERTION((aState.mCurrentLine == aLine), "oops");