Bug 163724 – [css-grid] Content Alignment broken with indefinite sized grid container

• Status: RESOLVED
• Resolution: FIXED
• Component: Layout and Rendering

[Javier Fernandez]

Created attachment 292172 [details]
Test case to reproduce the issue.

When the grid container has indefinite width or height the Content Alignment properties (align-content and justify-content) don't work as expected. 

It seems that we are not computing correctly the available free space when the container has indefinite size. That issue is affecting how we deal with Overflow Alignment on the align-content and justify-content properties.

In order to reproduce the issue, just load the attached test case and see how it behaves in the first grid, which is alignment using align-content: end.

[Javier Fernandez]

Created attachment 292173 [details]
Patch

[Manuel Rego Casasnovas]

Comment on attachment 292173 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=292173&action=review

r=me

> Source/WebCore/ChangeLog:8
> +        The Grid Tracks sizing algorithm updates receives the available space

Typo "updates receives".

> Source/WebCore/ChangeLog:19
> +        (WebCore::RenderGrid::layoutBlock): Compute freeSpace for Rows after doing layout and resolving the indefinite height.

Nit: You wrapped the lines in the rest of the ChangeLog comment but not here.

[Build Bot]

Comment on attachment 292173 [details]
Patch

Attachment 292173 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/2330882

New failing tests:
fast/css-grid-layout/grid-content-alignment-overflow.html

[Build Bot]

Created attachment 292179 [details]
Archive of layout-test-results from ews103 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews103  Port: mac-yosemite  Platform: Mac OS X 10.10.5

[Build Bot]

Comment on attachment 292173 [details]
Patch

Attachment 292173 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/2330893

New failing tests:
fast/css-grid-layout/grid-content-alignment-overflow.html

[Build Bot]

Created attachment 292181 [details]
Archive of layout-test-results from ews105 for mac-yosemite-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews105  Port: mac-yosemite-wk2  Platform: Mac OS X 10.10.5

[Build Bot]

Comment on attachment 292173 [details]
Patch

Attachment 292173 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/2330918

New failing tests:
fast/css-grid-layout/grid-content-alignment-overflow.html

[Build Bot]

Created attachment 292186 [details]
Archive of layout-test-results from ews123 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews123  Port: ios-simulator-wk2  Platform: Mac OS X 10.11.6

[Build Bot]

Comment on attachment 292173 [details]
Patch

Attachment 292173 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/2330944

New failing tests:
fast/css-grid-layout/grid-content-alignment-overflow.html

[Build Bot]

Created attachment 292187 [details]
Archive of layout-test-results from ews114 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews114  Port: mac-yosemite  Platform: Mac OS X 10.10.5

[Javier Fernandez]

Created attachment 292193 [details]
Patch

[Build Bot]

Comment on attachment 292193 [details]
Patch

Attachment 292193 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/2331607

New failing tests:
fast/css-grid-layout/grid-content-alignment-overflow.html

[Build Bot]

Created attachment 292204 [details]
Archive of layout-test-results from ews116 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews116  Port: mac-yosemite  Platform: Mac OS X 10.10.5

[Build Bot]

Comment on attachment 292193 [details]
Patch

Attachment 292193 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/2331682

New failing tests:
fast/css-grid-layout/grid-content-alignment-overflow.html

[Build Bot]

Created attachment 292205 [details]
Archive of layout-test-results from ews101 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews101  Port: mac-yosemite  Platform: Mac OS X 10.10.5

[Build Bot]

Comment on attachment 292193 [details]
Patch

Attachment 292193 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/2331731

New failing tests:
fast/css-grid-layout/grid-content-alignment-overflow.html

[Build Bot]

Created attachment 292209 [details]
Archive of layout-test-results from ews123 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews123  Port: ios-simulator-wk2  Platform: Mac OS X 10.11.6

[Build Bot]

Comment on attachment 292193 [details]
Patch

Attachment 292193 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/2331793

New failing tests:
fast/css-grid-layout/grid-content-alignment-overflow.html

[Build Bot]

Created attachment 292210 [details]
Archive of layout-test-results from ews105 for mac-yosemite-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews105  Port: mac-yosemite-wk2  Platform: Mac OS X 10.10.5

[Javier Fernandez]

Created attachment 292224 [details]
Patch

Fixed layout tests.

[WebKit Commit Bot]

Comment on attachment 292224 [details]
Patch

Clearing flags on attachment: 292224

Committed r207663: <http://trac.webkit.org/changeset/207663>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.