[css-grid] Use order-modified document order for m_gridItemsIndexesMap

[css-grid] Use order-modified document order for m_gridItemsIndexesMap

We were storing the index in the DOM for the grid items
in m_gridItemsIndexesMap. However, we can already store the index
following the order-modified document order.

This allows us to simplify the sorter in painting,
as we don't need to check the order property now,
just simply use the stored index.

No new tests, as it's already covered by current tests.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

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

[Manuel Rego, 2016-10-21 21:45:13]

Description was changed from

==========
[css-grid] Use order-modified document order for m_gridItemsIndexesMap

We were storing the index in the DOM for the grid items
in m_gridItemsIndexesMap. However, we can already store the index
following the order-modified document order.

This allows us to simplify the sorter in painting,
as we don't need to check the order property now,
just simply use the stored index.

No new tests, as it's already covered by current tests.
==========

to

==========
[css-grid] Use order-modified document order for m_gridItemsIndexesMap

We were storing the index in the DOM for the grid items
in m_gridItemsIndexesMap. However, we can already store the index
following the order-modified document order.

This allows us to simplify the sorter in painting,
as we don't need to check the order property now,
just simply use the stored index.

No new tests, as it's already covered by current tests.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[Manuel Rego, 2016-10-21 21:49:33]

Description was changed from

==========
[css-grid] Use order-modified document order for m_gridItemsIndexesMap

We were storing the index in the DOM for the grid items
in m_gridItemsIndexesMap. However, we can already store the index
following the order-modified document order.

This allows us to simplify the sorter in painting,
as we don't need to check the order property now,
just simply use the stored index.

No new tests, as it's already covered by current tests.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
[css-grid] Use order-modified document order for m_gridItemsIndexesMap

We were storing the index in the DOM for the grid items
in m_gridItemsIndexesMap. However, we can already store the index
following the order-modified document order.

This allows us to simplify the sorter in painting,
as we don't need to check the order property now,
just simply use the stored index.

No new tests, as it's already covered by current tests.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[Manuel Rego, 2016-10-21 21:50:32]

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

[Manuel Rego, 2016-10-21 21:50:33]

This is small refactoring as @jfernandez will need to use
this hash map for the grid baseline patch.

[cbiesinger, 2016-10-21 22:10:31]

lgtm

[Manuel Rego, 2016-10-21 22:15:44]

The CQ bit was checked by rego@igalia.com

[commit-bot: I haz the power, 2016-10-21 22:16:22]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-22 01:57:25]

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

[commit-bot: I haz the power, 2016-10-22 01:57:25]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Manuel Rego, 2016-10-22 06:35:16]

The CQ bit was checked by rego@igalia.com

[commit-bot: I haz the power, 2016-10-22 06:35:28]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-22 07:47:57]

Description was changed from

==========
[css-grid] Use order-modified document order for m_gridItemsIndexesMap

We were storing the index in the DOM for the grid items
in m_gridItemsIndexesMap. However, we can already store the index
following the order-modified document order.

This allows us to simplify the sorter in painting,
as we don't need to check the order property now,
just simply use the stored index.

No new tests, as it's already covered by current tests.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
[css-grid] Use order-modified document order for m_gridItemsIndexesMap

We were storing the index in the DOM for the grid items
in m_gridItemsIndexesMap. However, we can already store the index
following the order-modified document order.

This allows us to simplify the sorter in painting,
as we don't need to check the order property now,
just simply use the stored index.

No new tests, as it's already covered by current tests.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[commit-bot: I haz the power, 2016-10-22 07:47:58]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-22 07:50:01]

Description was changed from

==========
[css-grid] Use order-modified document order for m_gridItemsIndexesMap

We were storing the index in the DOM for the grid items
in m_gridItemsIndexesMap. However, we can already store the index
following the order-modified document order.

This allows us to simplify the sorter in painting,
as we don't need to check the order property now,
just simply use the stored index.

No new tests, as it's already covered by current tests.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
[css-grid] Use order-modified document order for m_gridItemsIndexesMap

We were storing the index in the DOM for the grid items
in m_gridItemsIndexesMap. However, we can already store the index
following the order-modified document order.

This allows us to simplify the sorter in painting,
as we don't need to check the order property now,
just simply use the stored index.

No new tests, as it's already covered by current tests.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

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

[commit-bot: I haz the power, 2016-10-22 07:50:02]

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