Bug 1108923 - Update comments to reflect the newly logicalized parameters. r=smontagu

layout/generic/nsHTMLReflowState.cpp

@@ -1901,11 +1901,11 @@ GetFlexContainer(nsIFrame* aFrame)
   return static_cast<nsFlexContainerFrame*>(parent);
 }
 
-// Flex items resolve percentage margin & padding against the flex
-// container's height (which is the containing block height).
+// Flex items resolve block-axis percentage margin & padding against the flex
+// container's block-size (which is the containing block block-size).
 // For everything else: the CSS21 spec requires that margin and padding
-// percentage values are calculated with respect to the *width* of the
-// containing block, even for margin & padding in the vertical axis.
+// percentage values are calculated with respect to the inline-size of the
+// containing block, even for margin & padding in the block axis.
 static nscoord
 BlockDirOffsetPercentBasis(const nsIFrame* aFrame,
                            nscoord aContainingBlockISize,

layout/generic/nsHTMLReflowState.h

@@ -180,13 +180,15 @@ private:
    * Computes margin values from the specified margin style information, and
    * fills in the mComputedMargin member.
    *
-   * @param aHorizontalPercentBasis
-   *    Length to use for resolving percentage margin values in the horizontal
-   *    axis. Usually the containing block width.
-   * @param aVerticalPercentBasis
-   *    Length to use for resolving percentage margin values in the vertical
-   *    axis.  Usually the containing block width, per CSS21 sec 8.3, but may
-   *    be the containing block *height*, e.g. in CSS3 Flexbox and Grid.
+   * @param aInlineDirPercentBasis
+   *    Length to use for resolving percentage margin values in the inline
+   *    axis. Usually the containing block inline-size (width if writing mode
+   *    is horizontal, and height if vertical).
+   * @param aBlockDirPercentBasis
+   *    Length to use for resolving percentage margin values in the block
+   *    axis.  Usually the containing block inline-size, per CSS21 sec 8.3
+   *    (read in conjunction with CSS Writing Modes sec 7.2), but may
+   *    be the containing block block-size, e.g. in CSS3 Flexbox and Grid.
    * @return true if the margin is dependent on the containing block size.
    */
   bool ComputeMargin(nscoord aInlineDirPercentBasis,
@@ -196,13 +198,14 @@ private:
    * Computes padding values from the specified padding style information, and
    * fills in the mComputedPadding member.
    *
-   * @param aHorizontalPercentBasis
-   *    Length to use for resolving percentage padding values in the horizontal
-   *    axis. Usually the containing block width.
-   * @param aVerticalPercentBasis
-   *    Length to use for resolving percentage padding values in the vertical
-   *    axis.  Usually the containing block width, per CSS21 sec 8.4, but may
-   *    be the containing block *height* in e.g. CSS3 Flexbox and Grid.
+   * @param aInlineDirPercentBasis
+   *    Length to use for resolving percentage padding values in the inline
+   *    axis. Usually the containing block inline-size.
+   * @param aBlockDirPercentBasis
+   *    Length to use for resolving percentage padding values in the block
+   *    axis.  Usually the containing block inline-size, per CSS21 sec 8.4,
+   *    but may be the containing block block-size in e.g. CSS3 Flexbox and
+   *    Grid.
    * @return true if the padding is dependent on the containing block size.
    */
    bool ComputePadding(nscoord aInlineDirPercentBasis,