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>
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   
Description Flags
Test case to reproduce the crash.
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]
Comment 2 Manuel Rego Casasnovas 2016-05-23 06:11:50 PDT
Comment on attachment 279562 [details]

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


> 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]
Comment 4 WebKit Commit Bot 2016-05-24 11:36:24 PDT
Comment on attachment 279672 [details]

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]

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.