Attachment 341195 Details for Bug 185936 – Patch

[patch] Patch

bug-185936-20180524091844.patch (text/plain), 6.30 KB, created by zalan on 2018-05-24 09:18:44 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: zalan

Created: 2018-05-24 09:18:44 PDT

Size: 6.30 KB

patch

obsolete

>Subversion Revision: 232152
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index d8ae730671004cdf1f166851563f6acd99af6bc5..27918bac55cec3c0e9bd22107f36edbfc7ed2ff8 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,16 @@
>+2018-05-24  Zalan Bujtas  <zalan@apple.com>
>+
>+        [LFC] Implement position computation for inflow positioned elements
>+        https://bugs.webkit.org/show_bug.cgi?id=185936
>+
>+        Reviewed by Antti Koivisto.
>+
>+        * layout/blockformatting/BlockFormattingContext.cpp:
>+        (WebCore::Layout::BlockFormattingContext::computeInFlowPositionedPosition const):
>+        * layout/blockformatting/BlockFormattingContext.h:
>+        * layout/blockformatting/BlockFormattingContextGeometry.cpp:
>+        (WebCore::Layout::BlockFormattingContext::Geometry::inFlowPositionedPosition):
>+
> 2018-05-24  Zalan Bujtas  <zalan@apple.com>
> 
>         [LFC] Implement FormattingContext::placeInFlowPositionedChildren
>diff --git a/Source/WebCore/layout/blockformatting/BlockFormattingContext.cpp b/Source/WebCore/layout/blockformatting/BlockFormattingContext.cpp
>index e1f67e72692e36244104e3b00cd471d65acb1688..1d978fe74e0482b2a391e82b47ea661418e8956d 100644
>--- a/Source/WebCore/layout/blockformatting/BlockFormattingContext.cpp
>+++ b/Source/WebCore/layout/blockformatting/BlockFormattingContext.cpp
>@@ -134,6 +134,12 @@ void BlockFormattingContext::computeStaticPosition(LayoutContext& layoutContext,
>     displayBox.setTopLeft(topLeft);
> }
> 
>+void BlockFormattingContext::computeInFlowPositionedPosition(LayoutContext& layoutContext, const Box& layoutBox, Display::Box& displayBox) const
>+{
>+    auto topLeft = Geometry::inFlowPositionedPosition(layoutContext, layoutBox);
>+    displayBox.setTopLeft(topLeft);
>+}
>+
> void BlockFormattingContext::computeInFlowHeight(LayoutContext& layoutContext, const Box& layoutBox, Display::Box& displayBox) const
> {
>     LayoutUnit computedHeight;
>diff --git a/Source/WebCore/layout/blockformatting/BlockFormattingContext.h b/Source/WebCore/layout/blockformatting/BlockFormattingContext.h
>index fed1a601a9befc95a9c282202c27b95acb5677c7..397828cd131e8b86a4f529acc31b4ab122bc5085 100644
>--- a/Source/WebCore/layout/blockformatting/BlockFormattingContext.h
>+++ b/Source/WebCore/layout/blockformatting/BlockFormattingContext.h
>@@ -52,6 +52,7 @@ public:
> 
> private:
>     void computeStaticPosition(LayoutContext&, const Box&, Display::Box&) const override;
>+    void computeInFlowPositionedPosition(LayoutContext&, const Box&, Display::Box&) const override;
>     void computeInFlowWidth(LayoutContext&, const Box&, Display::Box&) const override;
>     void computeInFlowHeight(LayoutContext&, const Box&, Display::Box&) const override;
> 
>@@ -65,6 +66,7 @@ private:
>         static LayoutUnit inFlowNonReplacedWidth(LayoutContext&, const Box&);
> 
>         static LayoutPoint staticPosition(LayoutContext&, const Box&);
>+        static LayoutPoint inFlowPositionedPosition(LayoutContext&, const Box&);
>     };
> };
> 
>diff --git a/Source/WebCore/layout/blockformatting/BlockFormattingContextGeometry.cpp b/Source/WebCore/layout/blockformatting/BlockFormattingContextGeometry.cpp
>index 6f4cfd1d94fc57504fb3bbe94a977c24ae9634b2..c671c66b42034d05c05a648312997717a20a2384 100644
>--- a/Source/WebCore/layout/blockformatting/BlockFormattingContextGeometry.cpp
>+++ b/Source/WebCore/layout/blockformatting/BlockFormattingContextGeometry.cpp
>@@ -137,6 +137,68 @@ LayoutPoint BlockFormattingContext::Geometry::staticPosition(LayoutContext& layo
>     return topLeft;
> }
> 
>+LayoutPoint BlockFormattingContext::Geometry::inFlowPositionedPosition(LayoutContext& layoutContext, const Box& layoutBox)
>+{
>+    ASSERT(layoutBox.isInFlowPositioned());
>+    // 9.4.3 Relative positioning
>+    //
>+    // The 'top' and 'bottom' properties move relatively positioned element(s) up or down without changing their size.
>+    // Top' moves the boxes down, and 'bottom' moves them up. Since boxes are not split or stretched as a result of 'top' or 'bottom', the used values are always: top = -bottom.
>+    //
>+    // 1. If both are 'auto', their used values are both '0'.
>+    // 2. If one of them is 'auto', it becomes the negative of the other.
>+    // 3. If neither is 'auto', 'bottom' is ignored (i.e., the used value of 'bottom' will be minus the value of 'top').
>+    auto& displayBox = *layoutContext.displayBoxForLayoutBox(layoutBox);
>+    auto& style = layoutBox.style();
>+
>+    auto top = style.logicalTop();
>+    auto bottom = style.logicalBottom();
>+    LayoutUnit topDelta;
>+
>+    if (top.isAuto() && bottom.isAuto()) {
>+        // #1
>+        topDelta = 0;
>+    } else if (top.isAuto()) {
>+        // #2
>+        topDelta = -bottom.value();
>+    } else if (bottom.isAuto()) {
>+        // #3 #4
>+        topDelta = top.value();
>+    }
>+
>+    // For relatively positioned elements, 'left' and 'right' move the box(es) horizontally, without changing their size.
>+    // 'Left' moves the boxes to the right, and 'right' moves them to the left.
>+    // Since boxes are not split or stretched as a result of 'left' or 'right', the used values are always: left = -right.
>+    //
>+    // 1. If both 'left' and 'right' are 'auto' (their initial values), the used values are '0' (i.e., the boxes stay in their original position).
>+    // 2. If 'left' is 'auto', its used value is minus the value of 'right' (i.e., the boxes move to the left by the value of 'right').
>+    // 3. If 'right' is specified as 'auto', its used value is minus the value of 'left'.
>+    // 4. If neither 'left' nor 'right' is 'auto', the position is over-constrained, and one of them has to be ignored.
>+    //    If the 'direction' property of the containing block is 'ltr', the value of 'left' wins and 'right' becomes -'left'.
>+    //    If 'direction' of the containing block is 'rtl', 'right' wins and 'left' is ignored.
>+    //
>+    auto left = style.logicalLeft();
>+    auto right = style.logicalRight();
>+    LayoutUnit leftDelta;
>+
>+    if (left.isAuto() && right.isAuto()) {
>+        // #1
>+        leftDelta = 0;
>+    } else if (left.isAuto()) {
>+        // #2
>+        leftDelta = -right.value();
>+    } else if (right.isAuto()) {
>+        // #3
>+        leftDelta = left.value();
>+    } else {
>+        // #4
>+        // FIXME: take direction into account
>+        leftDelta = left.value();
>+    }
>+
>+    return { displayBox.left() + leftDelta, displayBox.top() + topDelta };
>+}
>+
> }
> }
>

Actions: View | Formatted Diff | Diff

Attachments on bug 185936: 341172 | 341188 | 341195