tree 93be35f30c3881a93d1812b2121a197d614819e0
parent cc1a792689fa15026f7b0c85b1d8df715c813389
author Ana SollanoKim <ansollan@microsoft.com> 1625247488 -0700
committer Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> 1625248419 -0700

[GridNG] Placement of grid items with a negative margins

Previously, in |ContributionSizeForGridItem| margins were clamped to
zero before adding them to the contribution. Therefore, grid items with
negative margins were not being placed correctly. In this CL, the order
of operations is flipped: add the margin to the contribution and the
result is then clamped to zero.

Bug: 1045599, 1225424
Change-Id: I6fc2726d4744d520430b2ca96b3509a396d73538
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3002394
Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org>
Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#898232}