Bug 228920

Summary: [GLIB][cairo] Setting "overflow: clip" does not always clip overflow.
Product: WebKit Reporter: Arcady Goldmints-Orlov <crzwdjk>
Component: Layout and RenderingAssignee: Fujii Hironori <Hironori.Fujii>
Status: RESOLVED FIXED    
Severity: Normal CC: aperez, bfulgham, familyquerida, Hironori.Fujii, rbuis, simon.fraser, two, webkit-bug-importer, zalan
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Linux   
See Also: https://bugs.webkit.org/show_bug.cgi?id=198230
Attachments:
Description Flags
Screenshot of failling test
none
gardening patch none

Description Arcady Goldmints-Orlov 2021-08-09 12:54:12 PDT
Created attachment 435200 [details]
Screenshot of failling test

The tests imported/w3c/web-platform-tests/css/css-overflow/clip-006.html and imported/w3c/web-platform-tests/css/css-overflow/clip-007.html fail on both GTK and WPE, with the resulting images showing no green square when there is supposed to be a green square. Running the test directly from http://wpt.live does show a green square but the div containing the green square div has scroll bars which it is not supposed to. Attached is a scrrenshot from the wpt.live test.
Comment 1 Arcady Goldmints-Orlov 2021-08-09 13:02:06 PDT
Created attachment 435201 [details]
gardening patch
Comment 2 Rob Buis 2021-08-09 13:22:59 PDT
The problem IIRC was that the underlying rendering engine (cairo?) react strange to a clip path with large double minimum/maximum dimension, instead it seems to have some smaller min/max values. I don't think it is unreasonable to use double minimum/maximum so I think some clamping is missing in lower layers.
Comment 3 EWS 2021-08-09 13:45:11 PDT
Committed r280793 (240371@main): <https://commits.webkit.org/240371@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 435201 [details].
Comment 4 Simon Fraser (smfr) 2021-08-09 13:49:28 PDT
Underlying bug is not fixed.
Comment 5 Radar WebKit Bug Importer 2021-08-09 13:50:39 PDT
<rdar://problem/81711157>
Comment 6 Fujii Hironori 2023-05-10 13:01:00 PDT
*** Bug 248693 has been marked as a duplicate of this bug. ***
Comment 7 Fujii Hironori 2023-05-10 13:01:18 PDT
*** Bug 256139 has been marked as a duplicate of this bug. ***
Comment 8 Fujii Hironori 2023-05-10 13:22:49 PDT
Pull request: https://github.com/WebKit/WebKit/pull/13715
Comment 9 EWS 2023-05-10 19:28:59 PDT
Committed 263943@main (902804f38b45): <https://commits.webkit.org/263943@main>

Reviewed commits have been landed. Closing PR #13715 and removing active labels.