Bug 242000

Summary: Fix block-aspect-ratio-037.html
Product: WebKit Reporter: Rob Buis <rbuis>
Component: CSSAssignee: Rob Buis <rbuis>
Status: RESOLVED FIXED    
Severity: Normal CC: changseok, clopez, darin, esprehn+autocc, ews-watchlist, glenn, jochen.kuehner, kondapallykalyan, pdr, webkit-bug-importer, youennf, zalan
Priority: P2 Keywords: InRadar
Version: Safari 15   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

Description Rob Buis 2022-06-25 03:14:55 PDT
Fix block-aspect-ratio-037.html.
Comment 1 Rob Buis 2022-06-25 03:26:21 PDT
Created attachment 460490 [details]
Patch
Comment 2 EWS Watchlist 2022-06-25 03:28:22 PDT
This patch modifies the imported WPT tests. Please ensure that any changes on the tests (not coming from a WPT import) are exported to WPT. Please see https://trac.webkit.org/wiki/WPTExportProcess
Comment 3 Darin Adler 2022-06-26 16:54:26 PDT
Comment on attachment 460490 [details]
Patch

r=me

Is the single regression test sufficient coverage for both code changes?
Comment 4 Rob Buis 2022-06-27 06:21:35 PDT
Created attachment 460502 [details]
Patch
Comment 5 Rob Buis 2022-06-28 05:47:22 PDT
(In reply to Darin Adler from comment #3)
> Comment on attachment 460490 [details]
> Patch
> 
> r=me
> 
> Is the single regression test sufficient coverage for both code changes?

The change in constrainLogicalHeightByMinMax is covered by aspect-ratio/grid-aspect-ratio-022.html.
Comment 6 EWS 2022-06-28 05:54:27 PDT
Committed 251908@main (e97db329a49a): <https://commits.webkit.org/251908@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 460502 [details].
Comment 7 Radar WebKit Bug Importer 2022-06-28 05:55:12 PDT
<rdar://problem/96055491>
Comment 8 Rob Buis 2022-06-29 06:58:35 PDT
*** Bug 231775 has been marked as a duplicate of this bug. ***