Bug 149764 – [CSS Grid Layout] Don't need to reset auto-margins during grid items layout

• Status: RESOLVED
• Resolution: FIXED
• Component: Layout and Rendering

[Javier Fernandez]

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.

[Javier Fernandez]

Created attachment 262361 [details]
Patch

[Darin Adler]

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”.

[Javier Fernandez]

Created attachment 262524 [details]
Patch

[WebKit Commit Bot]

Comment on attachment 262524 [details]
Patch

Clearing flags on attachment: 262524

Committed r190633: <http://trac.webkit.org/changeset/190633>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.