Title: [286024] trunk/Source/WebCore
Revision
286024
Author
[email protected]
Date
2021-11-18 13:14:07 -0800 (Thu, 18 Nov 2021)

Log Message

[IFC][Integration] Switching line layout path should invalidate preferred width
https://bugs.webkit.org/show_bug.cgi?id=233324

Reviewed by Antti Koivisto.

Since we don't propagate intrinsic width values between the line layout codepaths, switching
should invalidate these dirty bits.

* rendering/RenderBlockFlow.cpp:
(WebCore::RenderBlockFlow::invalidateLineLayoutPath):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (286023 => 286024)


--- trunk/Source/WebCore/ChangeLog	2021-11-18 20:58:02 UTC (rev 286023)
+++ trunk/Source/WebCore/ChangeLog	2021-11-18 21:14:07 UTC (rev 286024)
@@ -1,3 +1,16 @@
+2021-11-18  Alan Bujtas  <[email protected]>
+
+        [IFC][Integration] Switching line layout path should invalidate preferred width
+        https://bugs.webkit.org/show_bug.cgi?id=233324
+
+        Reviewed by Antti Koivisto.
+
+        Since we don't propagate intrinsic width values between the line layout codepaths, switching
+        should invalidate these dirty bits.
+
+        * rendering/RenderBlockFlow.cpp:
+        (WebCore::RenderBlockFlow::invalidateLineLayoutPath):
+
 2021-11-18  Wenson Hsieh  <[email protected]>
 
         [macOS] [Live Text] Avoid analyzing images in editable content

Modified: trunk/Source/WebCore/rendering/RenderBlockFlow.cpp (286023 => 286024)


--- trunk/Source/WebCore/rendering/RenderBlockFlow.cpp	2021-11-18 20:58:02 UTC (rev 286023)
+++ trunk/Source/WebCore/rendering/RenderBlockFlow.cpp	2021-11-18 21:14:07 UTC (rev 286024)
@@ -3682,6 +3682,10 @@
         if (modernLineLayout() && modernLineLayout()->shouldSwitchToLegacyOnInvalidation())
             path = ForcedLegacyPath;
 #endif
+#if ENABLE_MODERN_PREFERRED_WIDTH_COMPUTATION
+        for (auto walker = InlineWalker(*this); !walker.atEnd(); walker.advance())
+            walker.current()->setPreferredLogicalWidthsDirty(true);
+#endif
         m_lineLayout = std::monostate();
         setLineLayoutPath(path);
         if (needsLayout())
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to