Title: [256097] trunk/Source/WebCore
Revision
256097
Author
[email protected]
Date
2020-02-08 18:01:13 -0800 (Sat, 08 Feb 2020)

Log Message

[LFC][BFC] Adjust Box::isFloatAvoider() for Inline and Independent FCs.
https://bugs.webkit.org/show_bug.cgi?id=207430
<rdar://problem/59288236>

Reviewed by Antti Koivisto.

A block container box that establishes both BFC and IFC is not a float avoider (add support for independent formatting context too).

* layout/layouttree/LayoutBox.cpp:
(WebCore::Layout::Box::isFloatAvoider const):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (256096 => 256097)


--- trunk/Source/WebCore/ChangeLog	2020-02-09 01:29:05 UTC (rev 256096)
+++ trunk/Source/WebCore/ChangeLog	2020-02-09 02:01:13 UTC (rev 256097)
@@ -1,3 +1,16 @@
+2020-02-08  Zalan Bujtas  <[email protected]>
+
+        [LFC][BFC] Adjust Box::isFloatAvoider() for Inline and Independent FCs.
+        https://bugs.webkit.org/show_bug.cgi?id=207430
+        <rdar://problem/59288236>
+
+        Reviewed by Antti Koivisto.
+
+        A block container box that establishes both BFC and IFC is not a float avoider (add support for independent formatting context too).
+
+        * layout/layouttree/LayoutBox.cpp:
+        (WebCore::Layout::Box::isFloatAvoider const):
+
 2020-02-08  Simon Fraser  <[email protected]>
 
         Extent of a composited animation should not include the untransformed position

Modified: trunk/Source/WebCore/layout/layouttree/LayoutBox.cpp (256096 => 256097)


--- trunk/Source/WebCore/layout/layouttree/LayoutBox.cpp	2020-02-09 01:29:05 UTC (rev 256096)
+++ trunk/Source/WebCore/layout/layouttree/LayoutBox.cpp	2020-02-09 02:01:13 UTC (rev 256097)
@@ -194,7 +194,8 @@
 
 bool Box::isFloatAvoider() const
 {
-    return establishesBlockFormattingContext() || establishesTableFormattingContext() || hasFloatClear();
+    return (establishesBlockFormattingContext() && !establishesInlineFormattingContext())
+        || establishesTableFormattingContext() || establishesIndependentFormattingContext() || hasFloatClear();
 }
 
 const Container* Box::containingBlock() const
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to