Bug 197849 – [css-grid] Grid not updated when changing auto repeat type

• Status: RESOLVED
• Resolution: FIXED
• Component: CSS

[Oriol Brufau]

What steps will reproduce the problem?
(1) Run this testcase:
```html
<style>
#grid {
  display: grid;
  width: 200px;
  background: linear-gradient(red, red) no-repeat 0 / 100px;
}
#item {
  grid-column-start: 2;
  height: 100px;
  background: green;
}
</style>
<div id="grid">
  <div id="item"></div>
</div>
<script>
var grid = document.getElementById("grid");
grid.style.gridTemplateColumns = "repeat(auto-fill, 100px)";
grid.style.gridTemplateColumns = "repeat(auto-fit, 100px)";
</script>
```

What is the expected result?
There should be a 100x100 green box with no red

What happens instead?
The green box appears more to the right, so you can see red.

This is because when WebKit compares `repeat(auto-fill, 100px)` and `repeat(auto-fit, 100px)`, it only checks whether the list of track sizes are equal, but it doesn't compare `auto-fill` and `auto-fit`. The former produces resolves to `100px 100px` and the latter to `0px 100px`, but Blink thinks they are equal so it doesn't update the grid.

This has been fixed in Blink: https://crbug.com/961407

[Oriol Brufau]

Created attachment 369766 [details]
Patch

[EWS Watchlist]

Comment on attachment 369766 [details]
Patch

Attachment 369766 [details] did not pass win-ews (win):
Output: https://webkit-queues.webkit.org/results/12187483

New failing tests:
imported/blink/compositing/layer-creation/iframe-clip-removed.html

[EWS Watchlist]

Created attachment 369842 [details]
Archive of layout-test-results from ews213 for win-future

The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews213  Port: win-future  Platform: CYGWIN_NT-10.0-17763-3.0.5-338.x86_64-x86_64-64bit

[Oriol Brufau]

Created attachment 369853 [details]
Patch

[Oriol Brufau]

Comment on attachment 369842 [details]
Archive of layout-test-results from ews213 for win-future

The EWS failure seems unrelated to my patch. Anyways, I have rebased it due to a conflict with bug 197854, so let's check again.

[Oriol Brufau]

Comment on attachment 369853 [details]
Patch

Win EWS is green now

[WebKit Commit Bot]

Comment on attachment 369853 [details]
Patch

Clearing flags on attachment: 369853

Committed r245295: <https://trac.webkit.org/changeset/245295>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/50776829>