Bug 157971

Summary: [css-grid] Horizontal scroll must account for grid container's height
Product: WebKit Reporter: Javier Fernandez <jfernandez>
Component: Layout and RenderingAssignee: Javier Fernandez <jfernandez>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, darin, esprehn+autocc, glenn, hyatt, jfernandez, kondapallykalyan, rego, simon.fraser, svillar
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Test case to reproduce the crash.
none
Patch
none
Patch
none
Archive of layout-test-results from webkit-cq-01 for mac-yosemite none

Description Javier Fernandez 2016-05-21 00:43:46 PDT
Created attachment 279540 [details]
Test case to reproduce the crash.

It seems grid layout logic does not consider the scrollbar size when computing the grid container's height. 
This may result in an incorrect sizing when using indefinite sized containers.

Besides, it's possible to make the layout logic to crash due to the fact that contentLogicalHeight can be a negative value if we render a case where content size is 0px.
Comment 1 Javier Fernandez 2016-05-23 05:24:20 PDT
Created attachment 279562 [details]
Patch
Comment 2 Manuel Rego Casasnovas 2016-05-23 06:11:50 PDT
Comment on attachment 279562 [details]
Patch

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

LGTM.

> LayoutTests/fast/css-grid-layout/grid-container-scroll-accounts-for-sizing-expected.html:7
> +   width: -webkit-fit-content;

Nit: Maybe you can use "css-intrinsic-dimensions/resources/width-keyword-classes.css".

Or simply something like "position: absolute" or "float: left" to force the shrink-to-fit.
Comment 3 Javier Fernandez 2016-05-24 11:00:32 PDT
Created attachment 279672 [details]
Patch
Comment 4 WebKit Commit Bot 2016-05-24 11:36:24 PDT
Comment on attachment 279672 [details]
Patch

Rejecting attachment 279672 [details] from commit-queue.

Number of test failures exceeded the failure limit.
Full output: http://webkit-queues.webkit.org/results/1375984
Comment 5 WebKit Commit Bot 2016-05-24 11:36:28 PDT
Created attachment 279678 [details]
Archive of layout-test-results from webkit-cq-01 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the commit-queue.
Bot: webkit-cq-01  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 6 WebKit Commit Bot 2016-06-06 05:09:57 PDT
Comment on attachment 279672 [details]
Patch

Clearing flags on attachment: 279672

Committed r201709: <http://trac.webkit.org/changeset/201709>
Comment 7 WebKit Commit Bot 2016-06-06 05:10:02 PDT
All reviewed patches have been landed.  Closing bug.