Title: [245437] releases/WebKitGTK/webkit-2.24
- Revision
- 245437
- Author
- [email protected]
- Date
- 2019-05-17 03:14:08 -0700 (Fri, 17 May 2019)
Log Message
Merge r243605 - [SimpleLineLayout] Disable SLL when text-underline-position is not auto.
https://bugs.webkit.org/show_bug.cgi?id=196338
<rdar://problem/47975167>
Reviewed by Daniel Bates.
Source/WebCore:
Disable simple line layout unconditionally on non-auto text-underline-position content. We don't support it yet.
Test: fast/text/simple-line-layout-with-text-underline-position.html
* rendering/SimpleLineLayout.cpp:
(WebCore::SimpleLineLayout::canUseForStyle):
LayoutTests:
* fast/text/simple-line-layout-with-text-underline-position-expected.html: Added.
* fast/text/simple-line-layout-with-text-underline-position.html: Added.
Modified Paths
Added Paths
Diff
Modified: releases/WebKitGTK/webkit-2.24/LayoutTests/ChangeLog (245436 => 245437)
--- releases/WebKitGTK/webkit-2.24/LayoutTests/ChangeLog 2019-05-17 10:14:03 UTC (rev 245436)
+++ releases/WebKitGTK/webkit-2.24/LayoutTests/ChangeLog 2019-05-17 10:14:08 UTC (rev 245437)
@@ -1,3 +1,14 @@
+2019-03-28 Zalan Bujtas <[email protected]>
+
+ [SimpleLineLayout] Disable SLL when text-underline-position is not auto.
+ https://bugs.webkit.org/show_bug.cgi?id=196338
+ <rdar://problem/47975167>
+
+ Reviewed by Daniel Bates.
+
+ * fast/text/simple-line-layout-with-text-underline-position-expected.html: Added.
+ * fast/text/simple-line-layout-with-text-underline-position.html: Added.
+
2019-03-11 Wenson Hsieh <[email protected]>
[macOS] Dispatching reentrant "contextmenu" events may cause crashes
Added: releases/WebKitGTK/webkit-2.24/LayoutTests/fast/text/simple-line-layout-with-text-underline-position-expected.html (0 => 245437)
--- releases/WebKitGTK/webkit-2.24/LayoutTests/fast/text/simple-line-layout-with-text-underline-position-expected.html (rev 0)
+++ releases/WebKitGTK/webkit-2.24/LayoutTests/fast/text/simple-line-layout-with-text-underline-position-expected.html 2019-05-17 10:14:08 UTC (rev 245437)
@@ -0,0 +1,7 @@
+<!DOCTYPE html>
+<html>
+<head>
+<title>This tests that simple line layout is disabled for text-underline-position: under</title>
+</head>
+<body></body>
+</html>
Added: releases/WebKitGTK/webkit-2.24/LayoutTests/fast/text/simple-line-layout-with-text-underline-position.html (0 => 245437)
--- releases/WebKitGTK/webkit-2.24/LayoutTests/fast/text/simple-line-layout-with-text-underline-position.html (rev 0)
+++ releases/WebKitGTK/webkit-2.24/LayoutTests/fast/text/simple-line-layout-with-text-underline-position.html 2019-05-17 10:14:08 UTC (rev 245437)
@@ -0,0 +1,28 @@
+<!DOCTYPE html>
+<html>
+<head>
+<title>This tests that simple line layout is disabled for text-underline-position: under</title>
+<style>
+div {
+ color: white;
+}
+
+.first {
+ overflow: hidden;
+}
+.second, .first {
+ text-underline-position: under;
+}
+</style>
+<script>
+if (window.internals) {
+ internals.settings.setSimpleLineLayoutDebugBordersEnabled(true);
+ internals.settings.setSimpleLineLayoutEnabled(true);
+}
+</script>
+</head>
+<body>
+<div class=first>Pass if after selecting these 2 lines</div>
+<div class=second>this line is still visible</div>
+</body>
+</html>
Modified: releases/WebKitGTK/webkit-2.24/Source/WebCore/ChangeLog (245436 => 245437)
--- releases/WebKitGTK/webkit-2.24/Source/WebCore/ChangeLog 2019-05-17 10:14:03 UTC (rev 245436)
+++ releases/WebKitGTK/webkit-2.24/Source/WebCore/ChangeLog 2019-05-17 10:14:08 UTC (rev 245437)
@@ -1,3 +1,18 @@
+2019-03-28 Zalan Bujtas <[email protected]>
+
+ [SimpleLineLayout] Disable SLL when text-underline-position is not auto.
+ https://bugs.webkit.org/show_bug.cgi?id=196338
+ <rdar://problem/47975167>
+
+ Reviewed by Daniel Bates.
+
+ Disable simple line layout unconditionally on non-auto text-underline-position content. We don't support it yet.
+
+ Test: fast/text/simple-line-layout-with-text-underline-position.html
+
+ * rendering/SimpleLineLayout.cpp:
+ (WebCore::SimpleLineLayout::canUseForStyle):
+
2019-03-11 Wenson Hsieh <[email protected]>
[macOS] Dispatching reentrant "contextmenu" events may cause crashes
Modified: releases/WebKitGTK/webkit-2.24/Source/WebCore/rendering/SimpleLineLayout.cpp (245436 => 245437)
--- releases/WebKitGTK/webkit-2.24/Source/WebCore/rendering/SimpleLineLayout.cpp 2019-05-17 10:14:03 UTC (rev 245436)
+++ releases/WebKitGTK/webkit-2.24/Source/WebCore/rendering/SimpleLineLayout.cpp 2019-05-17 10:14:08 UTC (rev 245437)
@@ -201,7 +201,7 @@
AvoidanceReasonFlags reasons = { };
if (style.textOverflow() == TextOverflow::Ellipsis)
SET_REASON_AND_RETURN_IF_NEEDED(FlowHasTextOverflow, reasons, includeReasons);
- if ((style.textDecorationsInEffect() & TextDecoration::Underline) && (style.textUnderlinePosition() != TextUnderlinePosition::Auto || !style.textUnderlineOffset().isAuto() || !style.textDecorationThickness().isAuto()))
+ if (style.textUnderlinePosition() != TextUnderlinePosition::Auto || !style.textUnderlineOffset().isAuto() || !style.textDecorationThickness().isAuto())
SET_REASON_AND_RETURN_IF_NEEDED(FlowHasUnsupportedUnderlineDecoration, reasons, includeReasons);
// Non-visible overflow should be pretty easy to support.
if (style.overflowX() != Overflow::Visible || style.overflowY() != Overflow::Visible)
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
