Bug 1232194 part 2 - [writing-mode] Make ConsumedBSize() return the block-axis size, not the physical height. r=dholbert

layout/generic/nsSplittableFrame.cpp

@@ -209,11 +209,11 @@ nsSplittableFrame::RemoveFromFlow(nsIFrame* aFrame)
 nscoord
 nsSplittableFrame::ConsumedBSize(WritingMode aWM) const
 {
-  nscoord height = 0;
+  nscoord bSize = 0;
   for (nsIFrame* prev = GetPrevInFlow(); prev; prev = prev->GetPrevInFlow()) {
-    height += prev->GetContentRectRelativeToSelf().height;
+    bSize += prev->ContentBSize(aWM);
   }
-  return height;
+  return bSize;
 }
 
 nscoord

layout/generic/nsSplittableFrame.h

@@ -81,7 +81,8 @@ protected:
   {}
 
   /**
-   * Determine the height consumed by our previous-in-flows.
+   * Return the sum of the block-axis content size of our prev-in-flows.
+   * @param aWM a writing-mode to determine the block-axis
    *
    * @note (bz) This makes laying out a splittable frame with N in-flows
    *       O(N^2)! So, use this function with caution and minimize the number