[css-grid] Fix positioned items with grid gaps

[css-grid] Fix positioned items with grid gaps

When we place a positioned items between in a grid with gaps,
we were not taking into accounts the gutter size.
We've to use that size to properly place and size the item.

Added new test checking the right behavior.

BUG=566265
TEST=fast/css-grid-layout/grid-positioned-items-gaps.html
TEST=fast/css-grid-layout/grid-positioned-items-gaps-rtl.html

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

[Manuel Rego, 2016-03-30 12:52:28]

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

[Manuel Rego, 2016-03-30 12:52:28]

This patch has been developed on top of:
http://crrev.com/1841073002/

So, please review the other one first. Thanks. :-)

[svillar, 2016-04-01 09:13:53]

Looks good. lgtm

Perhaps it'd be good to add a comment mentioning that vectors store positions
with gaps and that we should not consider them for the edges of the grid (or
something better written)

[Manuel Rego, 2016-04-01 10:44:13]

The CQ bit was checked by rego@igalia.com

[Manuel Rego, 2016-04-01 10:44:13]

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

[commit-bot: I haz the power, 2016-04-01 10:44:30]

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

[commit-bot: I haz the power, 2016-04-01 12:11:33]

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

[commit-bot: I haz the power, 2016-04-01 12:11:35]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Manuel Rego, 2016-04-01 14:22:50]

The CQ bit was checked by rego@igalia.com

[commit-bot: I haz the power, 2016-04-01 14:22:56]

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

[commit-bot: I haz the power, 2016-04-01 15:16:50]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-04-01 15:18:18]

Description was changed from

==========
[css-grid] Fix positioned items with grid gaps

When we place a positioned items between in a grid with gaps,
we were not taking into accounts the gutter size.
We've to use that size to properly place and size the item.

Added new test checking the right behavior.

BUG=566265
TEST=fast/css-grid-layout/grid-positioned-items-gaps.html
TEST=fast/css-grid-layout/grid-positioned-items-gaps-rtl.html
==========

to

==========
[css-grid] Fix positioned items with grid gaps

When we place a positioned items between in a grid with gaps,
we were not taking into accounts the gutter size.
We've to use that size to properly place and size the item.

Added new test checking the right behavior.

BUG=566265
TEST=fast/css-grid-layout/grid-positioned-items-gaps.html
TEST=fast/css-grid-layout/grid-positioned-items-gaps-rtl.html

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

[commit-bot: I haz the power, 2016-04-01 15:18:20]

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