Attachment 356256 Details for Bug 192264 – Patch

[patch] Patch

bug-192264-20181130160808.patch (text/plain), 8.47 KB, created by Myles C. Maxfield on 2018-11-30 16:08:09 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Myles C. Maxfield

Created: 2018-11-30 16:08:09 PST

Size: 8.47 KB

patch

obsolete

>Subversion Revision: 238751
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 1e7fd6a965309747a92e1557dc384e713c5f8f79..8868b6ae0f2639c0ed3caa42589434e95eba776a 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,19 @@
>+2018-11-30  Myles C. Maxfield  <mmaxfield@apple.com>
>+
>+        Thick overlines and line-throughs grow in the wrong direction
>+        https://bugs.webkit.org/show_bug.cgi?id=192264
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Overlines should grow upward, and line-throughs should stay centered.
>+
>+        Test: fast/css3-text/css3-text-decoration/text-decoration-thicknes-overline-grow-direction.html
>+
>+        * rendering/TextDecorationPainter.cpp:
>+        (WebCore::TextDecorationPainter::paintTextDecoration):
>+        * style/InlineTextBoxStyle.cpp:
>+        (WebCore::visualOverflowForDecorations):
>+
> 2018-11-30  Wenson Hsieh  <wenson_hsieh@apple.com>
> 
>         Replace "auto fill" with "AutoFill" in some localizable strings
>diff --git a/Source/WebCore/rendering/TextDecorationPainter.cpp b/Source/WebCore/rendering/TextDecorationPainter.cpp
>index 729cdefce0c215f6662d7efc157d74d104dce33f..7691cd0fe4867f5f21f2cc9922a2261e4b31d0cc 100644
>--- a/Source/WebCore/rendering/TextDecorationPainter.cpp
>+++ b/Source/WebCore/rendering/TextDecorationPainter.cpp
>@@ -296,12 +296,15 @@ void TextDecorationPainter::paintTextDecoration(const TextRun& textRun, const Fl
>         if (m_decorations.contains(TextDecoration::Overline)) {
>             float wavyOffset = m_styles.overlineStyle == TextDecorationStyle::Wavy ? m_wavyOffset : 0;
>             FloatRect rect(localOrigin, FloatSize(m_width, textDecorationThickness));
>-            rect.move(0, -wavyOffset);
>+            float autoTextDecorationThickness = TextDecorationThickness::createWithAuto().resolve(m_lineStyle.computedFontSize(), fontMetrics);
>+            rect.move(0, autoTextDecorationThickness - textDecorationThickness - wavyOffset);
>             paintDecoration(TextDecoration::Overline, m_styles.overlineStyle, m_styles.overlineColor, rect);
>         }
>         if (m_decorations.contains(TextDecoration::LineThrough)) {
>             FloatRect rect(localOrigin, FloatSize(m_width, textDecorationThickness));
>-            rect.move(0, 2 * fontMetrics.floatAscent() / 3);
>+            float autoTextDecorationThickness = TextDecorationThickness::createWithAuto().resolve(m_lineStyle.computedFontSize(), fontMetrics);
>+            auto center = 2 * fontMetrics.floatAscent() / 3 + autoTextDecorationThickness / 2;
>+            rect.move(0, center - textDecorationThickness / 2);
>             paintDecoration(TextDecoration::LineThrough, m_styles.linethroughStyle, m_styles.linethroughColor, rect);
>         }
>     } while (shadow);
>diff --git a/Source/WebCore/style/InlineTextBoxStyle.cpp b/Source/WebCore/style/InlineTextBoxStyle.cpp
>index 8b8174ac4e18db08362fa7cd5a9eab032d7a2d8a..878a586ed354105b405eb74febfa0e649062e730 100644
>--- a/Source/WebCore/style/InlineTextBoxStyle.cpp
>+++ b/Source/WebCore/style/InlineTextBoxStyle.cpp
>@@ -125,6 +125,7 @@ GlyphOverflow visualOverflowForDecorations(const RenderStyle& lineStyle, const I
>     }
> 
>     // These metrics must match where underlines get drawn.
>+    // FIXME: Share the code in TextDecorationPainter::paintTextDecoration() so we can just query it for the painted geometry.
>     if (decoration & TextDecoration::Underline) {
>         // Compensate for the integral ceiling in GraphicsContext::computeLineBoundsAndAntialiasingModeForText()
>         int underlineOffset = 1;
>@@ -140,23 +141,23 @@ GlyphOverflow visualOverflowForDecorations(const RenderStyle& lineStyle, const I
>         }
>     }
>     if (decoration & TextDecoration::Overline) {
>-        if (decorationStyle == TextDecorationStyle::Wavy) {
>-            extendIntToFloat(overflowResult.bottom, -wavyOffset + wavyStrokeParameters.controlPointDistance + strokeThickness - height);
>-            extendIntToFloat(overflowResult.top, wavyOffset + wavyStrokeParameters.controlPointDistance + strokeThickness);
>-        } else {
>-            extendIntToFloat(overflowResult.bottom, strokeThickness - height);
>-            // top is untouched
>-        }
>+        FloatRect rect(FloatPoint(), FloatSize(1, strokeThickness));
>+        float autoTextDecorationThickness = TextDecorationThickness::createWithAuto().resolve(lineStyle.computedFontSize(), lineStyle.fontMetrics());
>+        rect.move(0, autoTextDecorationThickness - strokeThickness - wavyOffset);
>+        if (decorationStyle == TextDecorationStyle::Wavy)
>+            rect.expand(FloatBoxExtent(wavyStrokeParameters.controlPointDistance, 0.f, wavyStrokeParameters.controlPointDistance, 0.f));
>+        extendIntToFloat(overflowResult.top, -rect.y());
>+        extendIntToFloat(overflowResult.bottom, rect.maxY());
>     }
>     if (decoration & TextDecoration::LineThrough) {
>-        float baseline = lineStyle.fontMetrics().floatAscent();
>-        if (decorationStyle == TextDecorationStyle::Wavy) {
>-            extendIntToFloat(overflowResult.bottom, 2 * baseline / 3 + wavyStrokeParameters.controlPointDistance + strokeThickness - height);
>-            extendIntToFloat(overflowResult.top, -(2 * baseline / 3 - wavyStrokeParameters.controlPointDistance - strokeThickness));
>-        } else {
>-            extendIntToFloat(overflowResult.bottom, 2 * baseline / 3 + strokeThickness - height);
>-            extendIntToFloat(overflowResult.top, -(2 * baseline / 3));
>-        }
>+        FloatRect rect(FloatPoint(), FloatSize(1, strokeThickness));
>+        float autoTextDecorationThickness = TextDecorationThickness::createWithAuto().resolve(lineStyle.computedFontSize(), lineStyle.fontMetrics());
>+        auto center = 2 * lineStyle.fontMetrics().floatAscent() / 3 + autoTextDecorationThickness / 2;
>+        rect.move(0, center - strokeThickness / 2);
>+        if (decorationStyle == TextDecorationStyle::Wavy)
>+            rect.expand(FloatBoxExtent(wavyStrokeParameters.controlPointDistance, 0.f, wavyStrokeParameters.controlPointDistance, 0.f));
>+        extendIntToFloat(overflowResult.top, -rect.y());
>+        extendIntToFloat(overflowResult.bottom, rect.maxY());
>     }
>     return overflowResult;
> }
>diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
>index a34df2f0716fbf4735ed917cc9c639a88b1ce4af..9bed85d8cac411ae6877e843ffb40c705d9c1758 100644
>--- a/LayoutTests/ChangeLog
>+++ b/LayoutTests/ChangeLog
>@@ -1,3 +1,15 @@
>+2018-11-30  Myles C. Maxfield  <mmaxfield@apple.com>
>+
>+        Thick overlines and line-throughs grow in the wrong direction
>+        https://bugs.webkit.org/show_bug.cgi?id=192264
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        I can't figure out a way to test the line-through, so this just tests the overline.
>+
>+        * fast/css3-text/css3-text-decoration/text-decoration-thicknes-overline-grow-direction-expected.html: Added.
>+        * fast/css3-text/css3-text-decoration/text-decoration-thicknes-overline-grow-direction.html: Added.
>+
> 2018-11-30  Wenson Hsieh  <wenson_hsieh@apple.com>
> 
>         Replace "auto fill" with "AutoFill" in some localizable strings
>diff --git a/LayoutTests/fast/css3-text/css3-text-decoration/text-decoration-thicknes-overline-grow-direction-expected.html b/LayoutTests/fast/css3-text/css3-text-decoration/text-decoration-thicknes-overline-grow-direction-expected.html
>new file mode 100644
>index 0000000000000000000000000000000000000000..5001c718424d53cc40044bebba4e552d6a72991c
>--- /dev/null
>+++ b/LayoutTests/fast/css3-text/css3-text-decoration/text-decoration-thicknes-overline-grow-direction-expected.html
>@@ -0,0 +1,9 @@
>+<!DOCTYPE html>
>+<html>
>+<head>
>+</head>
>+<body>
>+This test makes sure that overlines grow upward. The test passes if the overline on the text below has the default thickness.
>+<div style="font-size: 48px; text-decoration: overline;">Hello</div>
>+</body>
>+</html>
>diff --git a/LayoutTests/fast/css3-text/css3-text-decoration/text-decoration-thicknes-overline-grow-direction.html b/LayoutTests/fast/css3-text/css3-text-decoration/text-decoration-thicknes-overline-grow-direction.html
>new file mode 100644
>index 0000000000000000000000000000000000000000..c10f09b06eeede33247e7b3a96c5ca558a9aff0e
>--- /dev/null
>+++ b/LayoutTests/fast/css3-text/css3-text-decoration/text-decoration-thicknes-overline-grow-direction.html
>@@ -0,0 +1,9 @@
>+<!DOCTYPE html>
>+<html>
>+<head>
>+</head>
>+<body>
>+This test makes sure that overlines grow upward. The test passes if the overline on the text below has the default thickness.
>+<div style="font-size: 48px; text-decoration: overline; overflow: hidden; text-decoration-thickness: 10px;">Hello</div>
>+</body>
>+</html>

Actions: View | Formatted Diff | Diff

Attachments on bug 192264: 356256 | 356261 | 356276 | 356281 | 356283 | 356292 | 356299 | 356437 | 356553 | 356568