[css-grid] Avoid duplicated calls to resolution code

[css-grid] Avoid duplicated calls to resolution code

We were calling GridResolvedPosition::resolveGridPositionsFromStyle()
several times per item.

We can store the GridCoordinates in
LayoutGrid::populateExplicitGridAndOrderIterator()
and reuse them in the placement code.
Once LayoutGrid::placeItemsOnGrid() is over,
all the items will have a definite position in both axis.

No new tests, no change of behavior.

BUG=444011
GridResolvedPosition::resolveGridPositionsFromStyle

Committed: https://crrev.com/a7e24b5853a11308a12b302b34c9f624f2acd367
Cr-Commit-Position: refs/heads/master@{#361855}

[Manuel Rego, 2015-11-23 22:17:01]

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

[Manuel Rego, 2015-11-23 22:17:01]

Another small patch following the plan described at
http://code.google.com/p/chromium/issues/detail?id=444011#c7

[jfernandez, 2015-11-25 21:28:54]

LGTM, but I admit I'm not an expert on the placement logic.

[cbiesinger, 2015-11-26 01:17:43]

lgtm. looks reasonable.

[Manuel Rego, 2015-11-26 08:23:13]

The CQ bit was checked by rego@igalia.com

[commit-bot: I haz the power, 2015-11-26 08:23:24]

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

[commit-bot: I haz the power, 2015-11-26 09:32:27]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-11-26 09:33:26]

Description was changed from

==========
[css-grid] Avoid duplicated calls to resolution code

We were calling GridResolvedPosition::resolveGridPositionsFromStyle()
several times per item.

We can store the GridCoordinates in
LayoutGrid::populateExplicitGridAndOrderIterator()
and reuse them in the placement code.
Once LayoutGrid::placeItemsOnGrid() is over,
all the items will have a definite position in both axis.

No new tests, no change of behavior.

BUG=444011

GridResolvedPosition::resolveGridPositionsFromStyle
==========

to

==========
[css-grid] Avoid duplicated calls to resolution code

We were calling GridResolvedPosition::resolveGridPositionsFromStyle()
several times per item.

We can store the GridCoordinates in
LayoutGrid::populateExplicitGridAndOrderIterator()
and reuse them in the placement code.
Once LayoutGrid::placeItemsOnGrid() is over,
all the items will have a definite position in both axis.

No new tests, no change of behavior.

BUG=444011

GridResolvedPosition::resolveGridPositionsFromStyle

Committed: https://crrev.com/a7e24b5853a11308a12b302b34c9f624f2acd367
Cr-Commit-Position: refs/heads/master@{#361855}
==========

[commit-bot: I haz the power, 2015-11-26 09:33:27]

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