Modified: trunk/Source/WebCore/ChangeLog (281381 => 281382)
--- trunk/Source/WebCore/ChangeLog 2021-08-21 22:34:46 UTC (rev 281381)
+++ trunk/Source/WebCore/ChangeLog 2021-08-21 23:01:13 UTC (rev 281382)
@@ -1,3 +1,16 @@
+2021-08-21 Zalan Bujtas <za...@apple.com>
+
+ [IFC][Integration] Group non-standard CSS properties
+ https://bugs.webkit.org/show_bug.cgi?id=228855
+ <rdar://problem/81880442>
+
+ Reviewed by Antti Koivisto.
+
+ List of unsupported non-standard CSS properties (consider it a priority list).
+
+ * layout/integration/LayoutIntegrationCoverage.cpp:
+ (WebCore::LayoutIntegration::canUseForStyle):
+
2021-08-21 Per Arne <pvol...@apple.com>
[Win] Crash under FontCache::lastResortFallbackFont
Modified: trunk/Source/WebCore/layout/integration/LayoutIntegrationCoverage.cpp (281381 => 281382)
--- trunk/Source/WebCore/layout/integration/LayoutIntegrationCoverage.cpp 2021-08-21 22:34:46 UTC (rev 281381)
+++ trunk/Source/WebCore/layout/integration/LayoutIntegrationCoverage.cpp 2021-08-21 23:01:13 UTC (rev 281382)
@@ -510,8 +510,6 @@
SET_REASON_AND_RETURN_IF_NEEDED(FlowHasTextOverflow, reasons, includeReasons);
if (!style.isLeftToRightDirection())
SET_REASON_AND_RETURN_IF_NEEDED(FlowIsNotLTR, reasons, includeReasons);
- if (!(style.lineBoxContain().contains(LineBoxContain::Block)))
- SET_REASON_AND_RETURN_IF_NEEDED(FlowHasLineBoxContainProperty, reasons, includeReasons);
if (style.writingMode() != WritingMode::TopToBottom)
SET_REASON_AND_RETURN_IF_NEEDED(FlowIsNotTopToBottom, reasons, includeReasons);
if (style.unicodeBidi() != UBNormal)
@@ -518,10 +516,6 @@
SET_REASON_AND_RETURN_IF_NEEDED(FlowHasNonNormalUnicodeBiDi, reasons, includeReasons);
if (style.rtlOrdering() != Order::Logical)
SET_REASON_AND_RETURN_IF_NEEDED(FlowHasRTLOrdering, reasons, includeReasons);
- if (style.lineAlign() != LineAlign::None)
- SET_REASON_AND_RETURN_IF_NEEDED(FlowHasLineAlignEdges, reasons, includeReasons);
- if (style.lineSnap() != LineSnap::None)
- SET_REASON_AND_RETURN_IF_NEEDED(FlowHasLineSnap, reasons, includeReasons);
if (style.textEmphasisFill() != TextEmphasisFill::Filled || style.textEmphasisMark() != TextEmphasisMark::None)
SET_REASON_AND_RETURN_IF_NEEDED(FlowHasTextEmphasisFillOrMark, reasons, includeReasons);
if (style.textShadow())
@@ -534,14 +528,22 @@
SET_REASON_AND_RETURN_IF_NEEDED(FlowHasTextCombine, reasons, includeReasons);
if (style.backgroundClip() == FillBox::Text)
SET_REASON_AND_RETURN_IF_NEEDED(FlowHasTextFillBox, reasons, includeReasons);
- if (style.borderFit() == BorderFit::Lines)
- SET_REASON_AND_RETURN_IF_NEEDED(FlowHasBorderFitLines, reasons, includeReasons);
- if (style.lineBreak() == LineBreak::AfterWhiteSpace)
- SET_REASON_AND_RETURN_IF_NEEDED(FlowHasAfterWhiteSpaceLineBreak, reasons, includeReasons);
// Special handling of text-security:disc is not yet implemented in the simple line layout code path.
// See RenderBlock::updateSecurityDiscCharacters.
if (style.textSecurity() != TextSecurity::None)
SET_REASON_AND_RETURN_IF_NEEDED(FlowHasTextSecurity, reasons, includeReasons);
+
+ // These are non-standard properties.
+ if (style.lineBreak() == LineBreak::AfterWhiteSpace)
+ SET_REASON_AND_RETURN_IF_NEEDED(FlowHasAfterWhiteSpaceLineBreak, reasons, includeReasons);
+ if (!(style.lineBoxContain().contains(LineBoxContain::Block)))
+ SET_REASON_AND_RETURN_IF_NEEDED(FlowHasLineBoxContainProperty, reasons, includeReasons);
+ if (style.lineAlign() != LineAlign::None)
+ SET_REASON_AND_RETURN_IF_NEEDED(FlowHasLineAlignEdges, reasons, includeReasons);
+ if (style.lineSnap() != LineSnap::None)
+ SET_REASON_AND_RETURN_IF_NEEDED(FlowHasLineSnap, reasons, includeReasons);
+ if (style.borderFit() == BorderFit::Lines)
+ SET_REASON_AND_RETURN_IF_NEEDED(FlowHasBorderFitLines, reasons, includeReasons);
return reasons;
}
