[webkit-reviews] review granted: [Bug 197854] [css-grid] Use max size to compute auto repeat tracks : [Attachment 369790] Patch

[bugzilla-daemon at webkit.org]

Javier Fernandez <jfernandez at igalia.com> has granted Manuel Rego Casasnovas
<rego at igalia.com>'s request for review:
Bug 197854: [css-grid] Use max size to compute auto repeat tracks
https://bugs.webkit.org/show_bug.cgi?id=197854

Attachment 369790: Patch

https://bugs.webkit.org/attachment.cgi?id=369790&action=review




--- Comment #8 from Javier Fernandez <jfernandez at igalia.com> ---
Comment on attachment 369790
  --> https://bugs.webkit.org/attachment.cgi?id=369790
Patch

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

> Source/WebCore/rendering/RenderGrid.cpp:463
> +	   LayoutUnit availableMaxSize = LayoutUnit();

I don't think we need to initialize to LayoutUnit()

> Source/WebCore/rendering/RenderGrid.cpp:472
> +	   if (!availableMaxSize && !minSize.isSpecified())

I think the use of ! here is confusing (it may imply the availableMaxSize is an
optional value) Wouldn't be better (clearer) to use > 0 instead ?

> Source/WebCore/rendering/RenderGrid.cpp:475
> +	   LayoutUnit availableMinSize = LayoutUnit();

Ditto