When dealing with auto-margins alignment we were reseting auto values for margins, when grid items don't need to be laid out, just before using them to set the grid items location. This operation can be avoided if we just compute them, when required, during the execution of the stretching logic. This change could help us to avoid performing extra layout calls on some grid items.
Created attachment 262361 [details] Patch
Comment on attachment 262361 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=262361&action=review > Source/WebCore/rendering/RenderGrid.cpp:1505 > + const RenderStyle* parentStyle = &style(); Better to leave this as a reference and put the & at each call site. That helps us in the future when we change the functions to take references instead of pointers. > Source/WebCore/rendering/RenderGrid.cpp:1527 > + const RenderStyle* parentStyle = &style(); Ditto. > Source/WebCore/rendering/RenderGrid.cpp:1652 > + // In order to properly adjust the Self Alignment values we need to consider the offset between tracks. Not sure why it’s good to capitalize the words “Self Alignment”.
Created attachment 262524 [details] Patch
Comment on attachment 262524 [details] Patch Clearing flags on attachment: 262524 Committed r190633: <http://trac.webkit.org/changeset/190633>
All reviewed patches have been landed. Closing bug.