Fixed for-loop increments in InsertionPoint::setDistributedNodes.

Fixed for-loop increments in InsertionPoint::setDistributedNodes.

When we reached the end of the old/new distributed nodes in the inner
for-loop, we would do an extra increment of the index of the new/old
distributed nodes which would then lead to skipping reattach for one of
the elements.

Also removed the special treatment of reattaching fallback elements as
they would be part of the new/old vector and shouldn't need special
treatment.

The added test is currently not failing as it's hidden by a
SubtreeStyleChange on the host element which I'll try to remove in a
separate CL.

R=hayato@chromium.org,kochi@chromium.org,esprehn@chromium.org
BUG=584617
Committed: https://crrev.com/401f43989e320482084542f5f0153fcb7b4a8f98
Cr-Commit-Position: refs/heads/master@{#373805}

[rune, 2016-02-05 10:55:15]

The CQ bit was checked by rune@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-02-05 10:55:30]

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

[rune, 2016-02-05 10:55:31]

ptal

[commit-bot: I haz the power, 2016-02-05 12:13:08]

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

[commit-bot: I haz the power, 2016-02-05 12:13:09]

Dry run: This issue passed the CQ dry run.

[hayato, 2016-02-05 13:26:17]

lgtm

[rune, 2016-02-05 13:48:04]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2016-02-05 13:48:26]

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

[commit-bot: I haz the power, 2016-02-05 13:52:48]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-05 13:53:51]

Description was changed from

==========
Fixed for-loop increments in InsertionPoint::setDistributedNodes.

When we reached the end of the old/new distributed nodes in the inner
for-loop, we would do an extra increment of the index of the new/old
distributed nodes which would then lead to skipping reattach for one of
the elements.

Also removed the special treatment of reattaching fallback elements as
they would be part of the new/old vector and shouldn't need special
treatment.

The added test is currently not failing as it's hidden by a
SubtreeStyleChange on the host element which I'll try to remove in a
separate CL.

R=hayato@chromium.org,kochi@chromium.org,esprehn@chromium.org
BUG=584617
==========

to

==========
Fixed for-loop increments in InsertionPoint::setDistributedNodes.

When we reached the end of the old/new distributed nodes in the inner
for-loop, we would do an extra increment of the index of the new/old
distributed nodes which would then lead to skipping reattach for one of
the elements.

Also removed the special treatment of reattaching fallback elements as
they would be part of the new/old vector and shouldn't need special
treatment.

The added test is currently not failing as it's hidden by a
SubtreeStyleChange on the host element which I'll try to remove in a
separate CL.

R=hayato@chromium.org,kochi@chromium.org,esprehn@chromium.org
BUG=584617

Committed: https://crrev.com/401f43989e320482084542f5f0153fcb7b4a8f98
Cr-Commit-Position: refs/heads/master@{#373805}
==========

[commit-bot: I haz the power, 2016-02-05 13:53:52]

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