Bug 228920 - [GLIB] Setting "overflow: clip" does not always clip overflow.
Summary: [GLIB] Setting "overflow: clip" does not always clip overflow.
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Linux
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-08-09 12:54 PDT by Arcady Goldmints-Orlov
Modified: 2021-08-09 13:50 PDT (History)
6 users (show)

See Also:


Attachments
Screenshot of failling test (48.84 KB, image/png)
2021-08-09 12:54 PDT, Arcady Goldmints-Orlov
no flags Details
Patch (1.68 KB, patch)
2021-08-09 13:02 PDT, Arcady Goldmints-Orlov
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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]
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>