Created attachment 254173 [details] Example to reproduce the issue. If you render a grid and then you modify the height of a grid item (which previously had "height: auto;" and was stretched), the item is not resized as expected. If align-items is "start" you cannot reproduce the issue. Attached example to reproduce the issue, when you click on the button the "A" item should be resized to 100x100. Only width is resized.
Created attachment 254181 [details] Example to reproduce the issue.
Created attachment 254483 [details] Patch
Comment on attachment 254483 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=254483&action=review Nice catch! > Source/WebCore/rendering/RenderGrid.cpp:1300 > + child.clearOverrideLogicalContentHeight(); Don't we need to mark the child as needs layout in these cases?
(In reply to comment #3) > Comment on attachment 254483 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=254483&action=review > > Nice catch! > > > Source/WebCore/rendering/RenderGrid.cpp:1300 > > + child.clearOverrideLogicalContentHeight(); > > Don't we need to mark the child as needs layout in these cases? No, we don't. The overrideHeight is something we use to indicate, during layout, that instead of computing block's logical height a specific value must be used instead. Since grid layout uses this to set the stretched height, we need to clear it whenever stretching becomes not allowed. Whether block's logical height or not depends on many factors, what changed (width, margin, parent's height, ...) and that is not grid's layout responsibility; a block layout is forced only when its height is changed because of the stretching operation itself.
Comment on attachment 254483 [details] Patch Clearing flags on attachment: 254483 Committed r185327: <http://trac.webkit.org/changeset/185327>
All reviewed patches have been landed. Closing bug.