In RenderGrid::computePreferredLogicalWidths(), there is the following comment: // FIXME: We should account for min / max logical width. Other renderers (table, flex-box, ...) have some code to account for min-width / max-width when computing the preferred logical widths.
Created attachment 192947 [details] Proposed change 1: Handle min-width / max-width with testing.
Comment on attachment 192947 [details] Proposed change 1: Handle min-width / max-width with testing. Attachment 192947 [details] did not pass chromium-ews (chromium-xvfb): Output: http://webkit-commit-queue.appspot.com/results/17184044 New failing tests: fast/css-grid-layout/grid-element-empty-row-column.html fast/css-grid-layout/grid-preferred-logical-widths.html
Comment on attachment 192947 [details] Proposed change 1: Handle min-width / max-width with testing. View in context: https://bugs.webkit.org/attachment.cgi?id=192947&action=review > Source/WebCore/rendering/RenderGrid.cpp:196 > + for (size_t i = 0; i < m_grid.size(); ++i) { EWS <3 This is wrong m_grid.size() is the row count, not the column count. Will replace with gridColumnCount() which solves the failures. Also will add a sanity check in GridIterator that the starting indexes are inside the grid (which should always be the case as the grid has to be populated for GridIterator to be correct).
Comment on attachment 192947 [details] Proposed change 1: Handle min-width / max-width with testing. Attachment 192947 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-commit-queue.appspot.com/results/16997752 New failing tests: fast/css-grid-layout/grid-element-empty-row-column.html fast/css-grid-layout/grid-preferred-logical-widths.html
Comment on attachment 192947 [details] Proposed change 1: Handle min-width / max-width with testing. Attachment 192947 [details] did not pass mac-ews (mac): Output: http://webkit-commit-queue.appspot.com/results/17149506 New failing tests: fast/css-grid-layout/grid-element-empty-row-column.html fast/css-grid-layout/grid-preferred-logical-widths.html
Comment on attachment 192947 [details] Proposed change 1: Handle min-width / max-width with testing. Attachment 192947 [details] did not pass mac-ews (mac): Output: http://webkit-commit-queue.appspot.com/results/17149507 New failing tests: fast/css-grid-layout/grid-element-empty-row-column.html fast/css-grid-layout/grid-preferred-logical-widths.html
Created attachment 192989 [details] Patch for landing
Comment on attachment 192989 [details] Patch for landing Clearing flags on attachment: 192989 Committed r145758: <http://trac.webkit.org/changeset/145758>
All reviewed patches have been landed. Closing bug.