Bug 232176

Summary:	Improve performance of nested grids
Product:	WebKit	Reporter:	Tim Nguyen (:ntim) <ntim>
Component:	Layout and Rendering	Assignee:	Nobody <webkit-unassigned>
Status:	NEW ---
Severity:	Normal	CC:	agafvv, ahmad.saleem792, bfulgham, cathiechen, dobiasch, maggotfish, mattwoodrow, mendelgordons, michael.zacharov, rbuis, rego, rik, sgill26, simon.fraser, svillar, tobias47n9e, webkit-bug-importer, zalan, zsun
Priority:	P2	Keywords:	InRadar
Version:	WebKit Nightly Build
Hardware:	Unspecified
OS:	Unspecified
See Also:	https://bugs.webkit.org/show_bug.cgi?id=125145 https://bugs.webkit.org/show_bug.cgi?id=268595 https://bugs.webkit.org/show_bug.cgi?id=271083

Description Tim Nguyen (:ntim) 2021-10-22 14:21:30 PDT

See: http://exponential-layout.glitch.me

From: https://developer.chrome.com/blog/layoutng/

Comment 1 Radar WebKit Bug Importer 2021-10-29 14:22:19 PDT

<rdar://problem/84823492>

Comment 2 Ulrich Dobiasch 2022-09-23 06:17:49 PDT

Safari 16 – still present – 15+ nested grids and the cpu is stalling

Comment 3 Michael 2023-09-28 08:08:40 PDT

Safari 17 – still present – 15+ nested grids and the cpu is stalling

Comment 4 Mendel Gordon 2024-02-12 06:33:35 PST

See https://web.archive.org/web/20211008182612/https://developer.chrome.com/blog/layoutng/ since the original link is dead 
(In reply to Tim Nguyen (:ntim) from comment #0)
> See: http://exponential-layout.glitch.me
> 
> From: https://developer.chrome.com/blog/layoutng/

Comment 5 tobias47n9e@gmail.com 2024-02-12 08:12:01 PST

Just a comment regarding importance:

Would be nice to see this get higher priority. Grid is used for so many layout problems now, and the developer pain of having to remove all that just for Webkit based browsers is incredible.

Comment 6 Ahmad Saleem 2024-03-26 14:56:12 PDT

I tested this post 276633@main and now this test can be pushed to 100+ without any issue on M1 Pro with 16 GB.

Change progressing this - https://commits.webkit.org/276633@main

@Sammy - should we mark this duplicate of other bug?

Comment 7 Manuel Rego Casasnovas 2024-04-08 01:38:02 PDT

(In reply to Ahmad Saleem from comment #6)
> I tested this post 276633@main and now this test can be pushed to 100+
> without any issue on M1 Pro with 16 GB.
> 
> Change progressing this - https://commits.webkit.org/276633@main
> 
> @Sammy - should we mark this duplicate of other bug?

I can confirm this behavior, without that patch my computer starts to struggle around adding the 15th layer (or earlier). With that patch I've managed to add >150 layers easily.