[webkit-changes] [WebKit/WebKit] 1034f6: Applying wavy underline text decoration on <del> t...

Said Abou-Hallawa noreply at github.com
Tue Oct 31 15:44:46 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 1034f6e167610d43cd755fad80180c9c08ce23a4
      https://github.com/WebKit/WebKit/commit/1034f6e167610d43cd755fad80180c9c08ce23a4
  Author: Said Abou-Hallawa <said at apple.com>
  Date:   2023-10-31 (Tue, 31 Oct 2023)

  Changed paths:
    A LayoutTests/fast/text/del-text-zero-size-wavy-style-expected.txt
    A LayoutTests/fast/text/del-text-zero-size-wavy-style.html
    M LayoutTests/fast/text/text-zero-size-wavy-style.html
    M Source/WebCore/rendering/TextDecorationPainter.cpp

  Log Message:
  -----------
  Applying wavy underline text decoration on <del> text will hang if the font size is zero
https://bugs.webkit.org/show_bug.cgi?id=263989
rdar://113930569

Reviewed by Simon Fraser.

This should complete the intended fix of .

TextDecorationPainter::paintLineThrough() can still call strokeWavyTextDecoration()
even if the font size is zero. This part was missed from 269493 at main.

Like bug 260372, text within <del> tags and zero font size will cause WebKit to
hang and eventually jetsam by the system. The for-loop in strokeWavyTextDecoration()
will never terminate.

strokeWavyTextDecoration() should return if any of its parameters is zero.

* LayoutTests/fast/text/del-text-zero-size-wavy-style-expected.txt: Added.
* LayoutTests/fast/text/del-text-zero-size-wavy-style.html: Added.
* LayoutTests/fast/text/text-zero-size-wavy-style.html:
* Source/WebCore/rendering/TextDecorationPainter.cpp:
(WebCore::strokeWavyTextDecoration):

Canonical link: https://commits.webkit.org/270033@main




More information about the webkit-changes mailing list