Attachment 356553 Details for Bug 192264 – Patch for committing

[patch] Patch for committing

bug-192264-20181204164332.patch (text/plain), 8.70 KB, created by Myles C. Maxfield on 2018-12-04 16:43:33 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Myles C. Maxfield

Created: 2018-12-04 16:43:33 PST

Size: 8.70 KB

patch

obsolete

>Subversion Revision: 238872
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 9e7aeb8411f416d86d3b73c3bef828f5a9d8c6f3..c2f756dbd5f952530886a2afe862f6b369b6d5e4 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,19 @@
>+2018-12-04  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-12-04  Justin Michaud  <justin_michaud@apple.com>
> 
>         CSS Painting API should allow image values in inputProperties
>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..bed2843d8e7162acc0ae36c8a881e55af1adfd43 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,31 @@ GlyphOverflow visualOverflowForDecorations(const RenderStyle& lineStyle, const I
>         }
>     }
>     if (decoration & TextDecoration::Overline) {
>+        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) {
>-            extendIntToFloat(overflowResult.bottom, -wavyOffset + wavyStrokeParameters.controlPointDistance + strokeThickness - height);
>-            extendIntToFloat(overflowResult.top, wavyOffset + wavyStrokeParameters.controlPointDistance + strokeThickness);
>-        } else {
>-            extendIntToFloat(overflowResult.bottom, strokeThickness - height);
>-            // top is untouched
>+            FloatBoxExtent wavyExpansion;
>+            wavyExpansion.setTop(wavyStrokeParameters.controlPointDistance);
>+            wavyExpansion.setBottom(wavyStrokeParameters.controlPointDistance);
>+            rect.expand(wavyExpansion);
>         }
>+        extendIntToFloat(overflowResult.top, -rect.y());
>+        extendIntToFloat(overflowResult.bottom, rect.maxY() - height);
>     }
>     if (decoration & TextDecoration::LineThrough) {
>-        float baseline = lineStyle.fontMetrics().floatAscent();
>+        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) {
>-            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));
>+            FloatBoxExtent wavyExpansion;
>+            wavyExpansion.setTop(wavyStrokeParameters.controlPointDistance);
>+            wavyExpansion.setBottom(wavyStrokeParameters.controlPointDistance);
>+            rect.expand(wavyExpansion);
>         }
>+        extendIntToFloat(overflowResult.top, -rect.y());
>+        extendIntToFloat(overflowResult.bottom, rect.maxY() - height);
>     }
>     return overflowResult;
> }
>diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
>index 5494364d98e06db9aee55f8119eed2cf074b002a..23a744f14de0fed3ea45cd2e6656a63a1e597382 100644
>--- a/LayoutTests/ChangeLog
>+++ b/LayoutTests/ChangeLog
>@@ -1,3 +1,15 @@
>+2018-12-04  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-12-04  Justin Michaud  <justin_michaud@apple.com>
> 
>         CSS Painting API should allow image values in inputProperties
>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..680c6dd3c35126b6683c0823f7d383b4926c7def
>--- /dev/null
>+++ b/LayoutTests/fast/css3-text/css3-text-decoration/text-decoration-thicknes-overline-grow-direction-expected.html
>@@ -0,0 +1,10 @@
>+<!DOCTYPE html>
>+<html>
>+<head>
>+<meta name="viewport" content="width=device-width, initial-scale=1">
>+</head>
>+<body>
>+This test makes sure that overlines grow upward. The test passes if the top line below has the default thickness.
>+<div style="font: 48px 'Ahem'; text-decoration: overline;">pppp</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..ea3997fbb725b4880b4fdd8cb181dc6da4a7a443
>--- /dev/null
>+++ b/LayoutTests/fast/css3-text/css3-text-decoration/text-decoration-thicknes-overline-grow-direction.html
>@@ -0,0 +1,10 @@
>+<!DOCTYPE html>
>+<html>
>+<head>
>+<meta name="viewport" content="width=device-width, initial-scale=1">
>+</head>
>+<body>
>+This test makes sure that overlines grow upward. The test passes if the top line below has the default thickness.
>+<div style="font: 48px 'Ahem'; text-decoration: overline; overflow: hidden; text-decoration-thickness: 10px;">pppp</div>
>+</body>
>+</html>

Flags:

commit-queue: commit-queue-

Actions: View | Formatted Diff | Diff

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