[CSS Grid Layout] Avoid reset override logical width/height for non relative elements

[CSS Grid Layout] Avoid reset override logical width/height for non relative elements

We were setting the override logical width/height to -1 (no possible
resolution) for all the items regardless if they've a relative 
width/height or a fixed one.

Added the condition, including not only items with percentage logical
width/height but also relative, to avoid overriding the value for items 
with fixed width/height as it's not needed.

TEST=fast/css-grid-layout/grid-item-with-percent-height-in-auto-height-grid-resolution.html

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=191347

[Manuel Rego, 2015-02-20 15:35:24]

rego@igalia.com changed reviewers:
+ eae@chromium.org, jchaffraix@chromium.org, leviw@chromium.org

[Julien - ping for review, 2015-03-02 18:05:21]

lgtm

https://codereview.chromium.org/946553003/diff/20001/LayoutTests/fast/css-gri...
File
LayoutTests/fast/css-grid-layout/grid-item-with-percent-height-in-auto-height-grid-resolution.html
(right):

https://codereview.chromium.org/946553003/diff/20001/LayoutTests/fast/css-gri...
LayoutTests/fast/css-grid-layout/grid-item-with-percent-height-in-auto-height-grid-resolution.html:18:
min-height: 100%;
Let's add similar testing for max-width / max-height too.

[Manuel Rego, 2015-03-05 06:31:07]

Thanks for the review.

https://codereview.chromium.org/946553003/diff/20001/LayoutTests/fast/css-gri...
File
LayoutTests/fast/css-grid-layout/grid-item-with-percent-height-in-auto-height-grid-resolution.html
(right):

https://codereview.chromium.org/946553003/diff/20001/LayoutTests/fast/css-gri...
LayoutTests/fast/css-grid-layout/grid-item-with-percent-height-in-auto-height-grid-resolution.html:18:
min-height: 100%;
On 2015/03/02 18:05:20, Julien Chaffraix - PST wrote:
> Let's add similar testing for max-width / max-height too.

Done.

https://codereview.chromium.org/946553003/diff/20001/Source/core/layout/Layou...
File Source/core/layout/LayoutGrid.cpp (right):

https://codereview.chromium.org/946553003/diff/20001/Source/core/layout/Layou...
Source/core/layout/LayoutGrid.cpp:629:
child.setOverrideContainingBlockContentLogicalHeight(-1);
Applied the same change to the new lines doing:
  child.setOverrideContainingBlockContentLogicalWidth(-1);

Which were introduced in:
https://codereview.chromium.org/914993003

[Manuel Rego, 2015-03-05 06:31:13]

The CQ bit was checked by rego@igalia.com

[Manuel Rego, 2015-03-05 06:31:13]

The patchset sent to the CQ was uploaded after l-g-t-m from
jchaffraix@chromium.org
Link to the patchset: https://codereview.chromium.org/946553003/#ps40001 (title:
"Patch for landing")

[commit-bot: I haz the power, 2015-03-05 06:32:25]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/946553003/40001

[commit-bot: I haz the power, 2015-03-05 08:12:00]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=191347