Make sure to lazyReattach a distributed node if it is already attached

Make sure to lazyReattach a distributed node if it is already attached

For Shadow DOM v0, InsertionPoint::setDistributedNodes() is calling
lazyReattachIfAttached for distributed nodes, if necessary.

The same thing should be done for Shadow DOM v1 too. Unless that, the old layout
object will be re-used wrongly.

BUG=531990, 622185
Committed: https://crrev.com/505c030552ed7a13a108f77ce0829409e542b07c
Cr-Commit-Position: refs/heads/master@{#401512}

[hayato, 2016-06-22 10:28:38]

The CQ bit was checked by hayato@chromium.org to run a CQ dry run

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

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

[hayato, 2016-06-22 10:30:06]

Description was changed from

==========
Make sure to lazyReattach a distributed node if it is already attached

For Shadow DOM v0, InsertionPoint::setDistributedNodes() is calling
lazyReattachIfAttached for distributed nodes.
The same thing is neccessary to v1 too.

BUG=531990,622185
==========

to

==========
Make sure to lazyReattach a distributed node if it is already attached

For Shadow DOM v0, InsertionPoint::setDistributedNodes() is calling
lazyReattachIfAttached for distributed nodes, if necessary.

The same thing should be done for Shadow DOM v1 too.

BUG=531990,622185
==========

[hayato, 2016-06-22 10:38:45]

Description was changed from

==========
Make sure to lazyReattach a distributed node if it is already attached

For Shadow DOM v0, InsertionPoint::setDistributedNodes() is calling
lazyReattachIfAttached for distributed nodes, if necessary.

The same thing should be done for Shadow DOM v1 too.

BUG=531990,622185
==========

to

==========
Make sure to lazyReattach a distributed node if it is already attached

For Shadow DOM v0, InsertionPoint::setDistributedNodes() is calling
lazyReattachIfAttached for distributed nodes, if necessary.

The same thing should be done for Shadow DOM v1 too. Unless that, the same
layout object will be re-used wrongly.

BUG=531990,622185
==========

[hayato, 2016-06-22 10:39:11]

Description was changed from

==========
Make sure to lazyReattach a distributed node if it is already attached

For Shadow DOM v0, InsertionPoint::setDistributedNodes() is calling
lazyReattachIfAttached for distributed nodes, if necessary.

The same thing should be done for Shadow DOM v1 too. Unless that, the same
layout object will be re-used wrongly.

BUG=531990,622185
==========

to

==========
Make sure to lazyReattach a distributed node if it is already attached

For Shadow DOM v0, InsertionPoint::setDistributedNodes() is calling
lazyReattachIfAttached for distributed nodes, if necessary.

The same thing should be done for Shadow DOM v1 too. Unless that, the same
layout
object will be re-used wrongly.

BUG=531990,622185
==========

[hayato, 2016-06-22 10:39:20]

Description was changed from

==========
Make sure to lazyReattach a distributed node if it is already attached

For Shadow DOM v0, InsertionPoint::setDistributedNodes() is calling
lazyReattachIfAttached for distributed nodes, if necessary.

The same thing should be done for Shadow DOM v1 too. Unless that, the same
layout
object will be re-used wrongly.

BUG=531990,622185
==========

to

==========
Make sure to lazyReattach a distributed node if it is already attached

For Shadow DOM v0, InsertionPoint::setDistributedNodes() is calling
lazyReattachIfAttached for distributed nodes, if necessary.

The same thing should be done for Shadow DOM v1 too. Unless that, the old layout
object will be re-used wrongly.

BUG=531990,622185
==========

[hayato, 2016-06-22 10:41:38]

hayato@chromium.org changed reviewers:
+ kochi@chromium.org, tkent@chromium.org

[hayato, 2016-06-22 10:41:38]

PTAL.

As I and kochi@ chatted offline, this is another approach of 
https://codereview.chromium.org/2085853004, to avoid forceful lazyReattach.

[commit-bot: I haz the power, 2016-06-22 11:40:42]

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

[commit-bot: I haz the power, 2016-06-22 11:40:42]

Dry run: This issue passed the CQ dry run.

[kochi, 2016-06-23 01:10:31]

LGTM

Let's think more about optimization later.

[hayato, 2016-06-23 01:44:13]

The CQ bit was checked by hayato@chromium.org

[commit-bot: I haz the power, 2016-06-23 01:44:28]

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

[commit-bot: I haz the power, 2016-06-23 01:49:41]

Committed patchset #1 (id:1)

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

Description was changed from

==========
Make sure to lazyReattach a distributed node if it is already attached

For Shadow DOM v0, InsertionPoint::setDistributedNodes() is calling
lazyReattachIfAttached for distributed nodes, if necessary.

The same thing should be done for Shadow DOM v1 too. Unless that, the old layout
object will be re-used wrongly.

BUG=531990,622185
==========

to

==========
Make sure to lazyReattach a distributed node if it is already attached

For Shadow DOM v0, InsertionPoint::setDistributedNodes() is calling
lazyReattachIfAttached for distributed nodes, if necessary.

The same thing should be done for Shadow DOM v1 too. Unless that, the old layout
object will be re-used wrongly.

BUG=531990,622185

Committed: https://crrev.com/505c030552ed7a13a108f77ce0829409e542b07c
Cr-Commit-Position: refs/heads/master@{#401512}
==========

[commit-bot: I haz the power, 2016-06-23 01:53:11]

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