Attachment 340318 Details for Bug 185585 – Patch

[patch] Patch

bug-185585-20180514083229.patch (text/plain), 10.07 KB, created by zalan on 2018-05-14 08:32:31 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: zalan

Created: 2018-05-14 08:32:31 PDT

Size: 10.07 KB

patch

obsolete

>Subversion Revision: 231756
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 0640c1c397a2d8cf1848a3f2274134fd126dcdc0..657c538c090cd4f37b8aa370aa808896984976a7 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,20 @@
>+2018-05-14  Zalan Bujtas  <zalan@apple.com>
>+
>+        [LFC] Implement height computation for non-replaced out of flow elements.
>+        https://bugs.webkit.org/show_bug.cgi?id=185585
>+
>+        Reviewed by Antti Koivisto.
>+
>+        * layout/FormattingContext.cpp:
>+        (WebCore::Layout::FormattingContext::computeHeight const):
>+        (WebCore::Layout::FormattingContext::computeOutOfFlowHeight const):
>+        (WebCore::Layout::FormattingContext::layoutOutOfFlowDescendants const):
>+        (WebCore::Layout::FormattingContext::computeOutOfFlowNonReplacedHeight const):
>+        (WebCore::Layout::FormattingContext::computeHeightForBlockFormattingContextRootWithAutoHeight const):
>+        * layout/FormattingContext.h:
>+        * layout/blockformatting/BlockFormattingContext.h:
>+        * layout/displaytree/DisplayBox.h:
>+
> 2018-05-14  Zalan Bujtas  <zalan@apple.com>
> 
>         [LFC] Implement width computation for non-replaced out of flow elements.
>diff --git a/Source/WebCore/layout/FormattingContext.cpp b/Source/WebCore/layout/FormattingContext.cpp
>index 10f41f3b1fadd02ddc1abe7760886f70fc4d8651..8da70dee8e08a67ffe281fcf08f438dacb39154a 100644
>--- a/Source/WebCore/layout/FormattingContext.cpp
>+++ b/Source/WebCore/layout/FormattingContext.cpp
>@@ -72,7 +72,7 @@ void FormattingContext::computeWidth(LayoutContext& layoutContext, const Box& la
> void FormattingContext::computeHeight(LayoutContext& layoutContext, const Box& layoutBox, Display::Box& displayBox) const
> {
>     if (layoutBox.isOutOfFlowPositioned())
>-        return computeOutOfFlowHeight(layoutBox, displayBox);
>+        return computeOutOfFlowHeight(layoutContext, layoutBox, displayBox);
>     if (layoutBox.isFloatingPositioned())
>         return computeFloatingHeight(layoutBox, displayBox);
>     return computeInFlowHeight(layoutContext, layoutBox, displayBox);
>@@ -91,8 +91,13 @@ void FormattingContext::computeFloatingWidth(const Box&, Display::Box&) const
> {
> }
> 
>-void FormattingContext::computeOutOfFlowHeight(const Box&, Display::Box&) const
>+void FormattingContext::computeOutOfFlowHeight(LayoutContext& layoutContext, const Box& layoutBox, Display::Box& displayBox) const
> {
>+    if (!layoutBox.isReplaced()) {
>+        computeOutOfFlowNonReplacedHeight(layoutContext, layoutBox, displayBox);
>+        return;
>+    }
>+    ASSERT_NOT_REACHED();
> }
> 
> void FormattingContext::computeFloatingHeight(const Box&, Display::Box&) const
>@@ -138,10 +143,95 @@ void FormattingContext::layoutOutOfFlowDescendants(LayoutContext& layoutContext)
>         auto formattingContext = layoutContext.formattingContext(layoutBox);
>         formattingContext->layout(layoutContext, layoutContext.establishedFormattingState(layoutBox, *formattingContext));
> 
>-        computeOutOfFlowHeight(layoutBox, displayBox);
>+        computeOutOfFlowHeight(layoutContext, layoutBox, displayBox);
>     }
> }
> 
>+void FormattingContext::computeOutOfFlowNonReplacedHeight(LayoutContext& layoutContext, const Box& layoutBox, Display::Box& displayBox) const
>+{
>+    // For absolutely positioned elements, the used values of the vertical dimensions must satisfy this constraint:
>+    // 'top' + 'margin-top' + 'border-top-width' + 'padding-top' + 'height' + 'padding-bottom' + 'border-bottom-width' + 'margin-bottom' + 'bottom'
>+    // = height of containing block
>+
>+    // If all three of 'top', 'height', and 'bottom' are auto, set 'top' to the static position and apply rule number three below.
>+
>+    // If none of the three are 'auto': If both 'margin-top' and 'margin-bottom' are 'auto', solve the equation under the extra
>+    // constraint that the two margins get equal values. If one of 'margin-top' or 'margin-bottom' is 'auto', solve the equation for that value.
>+    // If the values are over-constrained, ignore the value for 'bottom' and solve for that value.
>+
>+    // Otherwise, pick the one of the following six rules that applies.
>+
>+    // 1. 'top' and 'height' are 'auto' and 'bottom' is not 'auto', then the height is based on the content per 10.6.7,
>+    //     set 'auto' values for 'margin-top' and 'margin-bottom' to 0, and solve for 'top'
>+    // 2. 'top' and 'bottom' are 'auto' and 'height' is not 'auto', then set 'top' to the static position, set 'auto' values for
>+    //    'margin-top' and 'margin-bottom' to 0, and solve for 'bottom'
>+    // 3. 'height' and 'bottom' are 'auto' and 'top' is not 'auto', then the height is based on the content per 10.6.7, set 'auto'
>+    //     values for 'margin-top' and 'margin-bottom' to 0, and solve for 'bottom'
>+    // 4. 'top' is 'auto', 'height' and 'bottom' are not 'auto', then set 'auto' values for 'margin-top' and 'margin-bottom' to 0, and solve for 'top'
>+    // 5. 'height' is 'auto', 'top' and 'bottom' are not 'auto', then 'auto' values for 'margin-top' and 'margin-bottom' are set to 0 and solve for 'height'
>+    // 6. 'bottom' is 'auto', 'top' and 'height' are not 'auto', then set 'auto' values for 'margin-top' and 'margin-bottom' to 0 and solve for 'bottom'
>+    auto& style = layoutBox.style();
>+    auto top = style.logicalTop();
>+    auto bottom = style.logicalBottom();
>+    auto height = style.logicalHeight(); 
>+
>+    auto containingBlockHeight = layoutContext.displayBoxForLayoutBox(*layoutBox.containingBlock())->height();
>+    LayoutUnit computedHeightValue;
>+
>+    if ((top.isAuto() && height.isAuto() && bottom.isAuto())
>+        || (top.isAuto() && height.isAuto() && !bottom.isAuto())
>+        || (!top.isAuto() && height.isAuto() && bottom.isAuto())) {
>+        // All auto (#3), #1 and #3
>+        computedHeightValue = contentHeightForFormattingContextRoot(layoutContext, layoutBox);
>+    } else if (!top.isAuto() && height.isAuto() && !bottom.isAuto()) {
>+        // #5
>+        auto marginTop = style.marginTop().isAuto() ? LayoutUnit() : valueForLength(style.marginTop(), containingBlockHeight);
>+        auto marginBottom = style.marginBottom().isAuto() ? LayoutUnit() : valueForLength(style.marginBottom(), containingBlockHeight);
>+    
>+        auto containingBlockWidth = layoutContext.displayBoxForLayoutBox(*layoutBox.containingBlock())->width();
>+        auto paddingTop = valueForLength(style.paddingTop(), containingBlockWidth);
>+        auto paddingBottom = valueForLength(style.paddingBottom(), containingBlockWidth);
>+
>+        auto borderTopWidth = style.borderBeforeWidth();
>+        auto borderBottomWidth = style.borderAfterWidth();
>+
>+        computedHeightValue = containingBlockHeight - (top.value() + marginTop + borderTopWidth + paddingTop + paddingBottom + borderBottomWidth + marginBottom + bottom.value());
>+    } else if (!height.isAuto())
>+        computedHeightValue = valueForLength(height, containingBlockHeight);
>+    else
>+        ASSERT_NOT_REACHED();
>+
>+    displayBox.setHeight(computedHeightValue);
>+}
>+
>+LayoutUnit FormattingContext::contentHeightForFormattingContextRoot(LayoutContext& layoutContext, const Box& layoutBox) const
>+{
>+    ASSERT(layoutBox.style().logicalHeight().isAuto());
>+    if (!is<Container>(layoutBox) || !downcast<Container>(layoutBox).hasInFlowOrFloatingChild())
>+        return 0;
>+
>+    auto& formattingRootContainer = downcast<Container>(layoutBox);
>+    // 10.6.7 'Auto' heights for block formatting context roots
>+
>+    // If it only has inline-level children, the height is the distance between the top of the topmost line box and the bottom of the bottommost line box.
>+    // If it has block-level children, the height is the distance between the top margin-edge of the topmost block-level
>+    // child box and the bottom margin-edge of the bottommost block-level child box.
>+
>+    // In addition, if the element has any floating descendants whose bottom margin edge is below the element's bottom content edge,
>+    // then the height is increased to include those edges. Only floats that participate in this block formatting context are taken
>+    // into account, e.g., floats inside absolutely positioned descendants or other floats are not.
>+    if (formattingRootContainer.establishesInlineFormattingContext())
>+        return 0;
>+
>+    auto* firstDisplayBox = layoutContext.displayBoxForLayoutBox(*formattingRootContainer.firstInFlowChild());
>+    auto* lastDisplayBox = layoutContext.displayBoxForLayoutBox(*formattingRootContainer.lastInFlowChild());
>+
>+    auto top = firstDisplayBox->marginBox().y();
>+    auto bottom = lastDisplayBox->marginBox().maxY();
>+    // FIXME: add floating support.
>+    return bottom - top;
>+}
>+
> void FormattingContext::computeOutOfFlowNonReplacedWidth(LayoutContext& layoutContext, const Box& layoutBox, Display::Box& displayBox) const
> {
>     // 'left' + 'margin-left' + 'border-left-width' + 'padding-left' + 'width' + 'padding-right' + 'border-right-width' + 'margin-right' + 'right'
>diff --git a/Source/WebCore/layout/FormattingContext.h b/Source/WebCore/layout/FormattingContext.h
>index ba83b85580bd3118fa0a778c4ca8eda3f8f1c536..42faedaff77735f46a663c6953419278f4b1ed14 100644
>--- a/Source/WebCore/layout/FormattingContext.h
>+++ b/Source/WebCore/layout/FormattingContext.h
>@@ -75,7 +75,7 @@ protected:
>     virtual void computeFloatingWidth(const Box&, Display::Box&) const;
>     virtual void computeInFlowWidth(const Box&, Display::Box&) const = 0;
> 
>-    virtual void computeOutOfFlowHeight(const Box&, Display::Box&) const;
>+    virtual void computeOutOfFlowHeight(LayoutContext&, const Box&, Display::Box&) const;
>     virtual void computeFloatingHeight(const Box&, Display::Box&) const;
>     virtual void computeInFlowHeight(LayoutContext&, const Box&, Display::Box&) const = 0;
> 
>@@ -88,7 +88,10 @@ protected:
>     void layoutOutOfFlowDescendants(LayoutContext&s) const;
> 
> private:
>+    void computeOutOfFlowNonReplacedHeight(LayoutContext&, const Box&, Display::Box&) const;
>     void computeOutOfFlowNonReplacedWidth(LayoutContext&, const Box&, Display::Box&) const;
>+
>+    LayoutUnit contentHeightForFormattingContextRoot(LayoutContext&, const Box&) const;
>     LayoutUnit shrinkToFitWidth(LayoutContext&, const Box&) const;
> 
>     WeakPtr<Box> m_root;

Actions: View | Formatted Diff | Diff

Attachments on bug 185585: 340255 | 340278 | 340282 | 340318