Bug 149086

Summary: Whitespace underlines are not repainted on color changes
Product: WebKit Reporter: Philip Rogers <pdr>
Component: CSSAssignee: Myles C. Maxfield <mmaxfield>
Status: RESOLVED FIXED    
Severity: Normal CC: ahmad.saleem792, bfulgham, Hironori.Fujii, karlcow, kling, mmaxfield, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Repro for webkit none

Description Philip Rogers 2015-09-11 21:22:05 PDT 
Created attachment 261042 [details]
Repro for webkit

wkrev.com/150259 introduced a small regression where text decorations such as underlines are not repainted for whitespace text.

A user recently reported this against Chrome and I had merged the above patch a few years ago so we shared this bug. I fixed this by removing the whitespace check which keeps the original optimization but causes whitespace runs to be repainted. My fix landed in https://src.chromium.org/viewvc/blink?revision=202164&view=revision and a similar fix would work in WebKit too.
Comment 1 Myles C. Maxfield 2015-09-12 21:13:56 PDT 
This repro is fantastic, thank you!
Comment 2 Brent Fulgham 2022-07-14 14:44:40 PDT 
WebKit still gets this wrong, 7 years later.
Comment 3 Radar WebKit Bug Importer 2022-07-14 14:44:49 PDT 
<rdar://problem/97036299>
Comment 4 Ahmad Saleem 2022-10-16 10:59:44 PDT 
I think this change should be here:

https://github.com/WebKit/WebKit/blob/main/Source/WebCore/rendering/RenderElement.cpp#L319

Replacing "isAllCollapsibleWhitespace" with equivalent of "isBR".
Comment 5 Fujii Hironori 2024-01-16 17:27:44 PST 
This is not reproducible with Safari 17.2.1 and STP 186 to me.
Comment 6 Ahmad Saleem 2024-01-16 17:31:55 PST 
(In reply to Fujii Hironori from comment #5)
> This is not reproducible with Safari 17.2.1 and STP 186 to me.

Yes - I can't reproduce it as well. Might be fixed between 2022 to 2024 now.
Comment 7 Fujii Hironori 2024-01-16 17:33:30 PST 
Thank you for the confirmation. Closed.