[css-grid] Avoid unneeded calls to percentageLogicalHeightIsResolvable()

[css-grid] Avoid unneeded calls to percentageLogicalHeightIsResolvable()

r398535 was introducing a performance regression on grid layout tests.
The problem is that it was calling LayoutGrid::hasDefiniteLogicalSize()
which was very slow.

r398680 was getting rid of hasDefiniteLogicalSize() and replacing it
by calls to LayoutBox::percentageLogicalHeightIsResolvable().
Despite being faster it's still an expensive operation and
we don't need to do it always.

So in this patch we're avoid calling percentageLogicalHeightIsResolvable()
when the min and max track breadths are not percentages.

BUG=618664, 616716
Committed: https://crrev.com/06fdd11aadb98dd6e25c6aaace831998b23eae86
Cr-Commit-Position: refs/heads/master@{#399130}

[Manuel Rego, 2016-06-09 21:02:05]

rego@igalia.com changed reviewers:
+ cbiesinger@chromium.org, jfernandez@igalia.com, svillar@igalia.com

[Manuel Rego, 2016-06-09 21:02:06]

This CL should be fixing the perf regression.
It's the same that fixed the regression in the past:
https://codereview.chromium.org/2037383002

The trybots are checking it at:
https://codereview.chromium.org/2050043005

[cbiesinger, 2016-06-09 22:50:20]

lgtm

[Manuel Rego, 2016-06-10 05:26:30]

The CQ bit was checked by rego@igalia.com

[commit-bot: I haz the power, 2016-06-10 05:26:40]

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

[commit-bot: I haz the power, 2016-06-10 06:34:53]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-06-10 06:34:54]

Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[Manuel Rego, 2016-06-10 06:56:39]

The CQ bit was checked by rego@igalia.com

[commit-bot: I haz the power, 2016-06-10 06:56:46]

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

[commit-bot: I haz the power, 2016-06-10 07:24:05]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-10 07:24:11]

CQ bit was unchecked

[commit-bot: I haz the power, 2016-06-10 07:25:11]

Description was changed from

==========
[css-grid] Avoid unneeded calls to percentageLogicalHeightIsResolvable()

r398535 was introducing a performance regression on grid layout tests.
The problem is that it was calling LayoutGrid::hasDefiniteLogicalSize()
which was very slow.

r398680 was getting rid of hasDefiniteLogicalSize() and replacing it
by calls to LayoutBox::percentageLogicalHeightIsResolvable().
Despite being faster it's still an expensive operation and
we don't need to do it always.

So in this patch we're avoid calling percentageLogicalHeightIsResolvable()
when the min and max track breadths are not percentages.

BUG=618664,616716
==========

to

==========
[css-grid] Avoid unneeded calls to percentageLogicalHeightIsResolvable()

r398535 was introducing a performance regression on grid layout tests.
The problem is that it was calling LayoutGrid::hasDefiniteLogicalSize()
which was very slow.

r398680 was getting rid of hasDefiniteLogicalSize() and replacing it
by calls to LayoutBox::percentageLogicalHeightIsResolvable().
Despite being faster it's still an expensive operation and
we don't need to do it always.

So in this patch we're avoid calling percentageLogicalHeightIsResolvable()
when the min and max track breadths are not percentages.

BUG=618664,616716

Committed: https://crrev.com/06fdd11aadb98dd6e25c6aaace831998b23eae86
Cr-Commit-Position: refs/heads/master@{#399130}
==========

[commit-bot: I haz the power, 2016-06-10 07:25:12]

Patchset 1 (id:??) landed as
https://crrev.com/06fdd11aadb98dd6e25c6aaace831998b23eae86
Cr-Commit-Position: refs/heads/master@{#399130}