Bug 229140

Summary: Incorrect repaint when inline level box style change triggers line height change
Product: WebKit Reporter: Tim Nguyen (:ntim) <ntim>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, changseok, esprehn+autocc, ews-watchlist, glenn, koivisto, kondapallykalyan, pdr, simon.fraser, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Testcase
none
Test reduction
none
Test reduction
none
patch none

Description Tim Nguyen (:ntim) 2021-08-16 07:56:54 PDT 
Created attachment 435602 [details]
Testcase

hi is displayed twice with the testcase.
Comment 1 Radar WebKit Bug Importer 2021-08-16 07:58:37 PDT 
<rdar://problem/81980863>
Comment 2 zalan 2021-08-16 08:08:56 PDT 
Created attachment 435603 [details]
Test reduction
Comment 3 zalan 2021-08-16 08:11:10 PDT 
Created attachment 435604 [details]
Test reduction
Comment 4 zalan 2021-08-16 08:13:03 PDT 
This looks to be timing related too. If you can't repro, try locally (or change the timeout).
Comment 5 Antti Koivisto 2021-08-16 09:51:47 PDT 
We fail to repaint the old content area when the block shrinks vertically as a result of a layout. Legacy handles this by invalidating deleted line boxes.

In many real world cases the content below moves, triggering repaint and hiding the issue.
Comment 6 Antti Koivisto 2021-08-17 01:29:00 PDT 
Created attachment 435671 [details]
patch
Comment 7 EWS 2021-08-17 07:11:30 PDT 
Committed r281136 (240590@main): <https://commits.webkit.org/240590@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 435671 [details].