Clarify that Blink does not support a re-distribution across v0 and v1 shadow trees

Clarify that Blink does not support a re-distribution across v0 and v1 shadow trees

I do not have a plan to support a re-distribution across v0 and v1 shadow trees.
It would be better to clarify it in the comment.

BUG=531990
Committed: https://crrev.com/42739d8adae310a94b87f153878a0f1d4365618b
Cr-Commit-Position: refs/heads/master@{#386606}

[hayato, 2016-04-11 09:27:21]

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

[commit-bot: I haz the power, 2016-04-11 09:27:36]

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

[hayato, 2016-04-11 09:30:29]

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

[hayato, 2016-04-11 09:30:30]

clarify

[commit-bot: I haz the power, 2016-04-11 09:30:44]

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

[hayato, 2016-04-11 09:32:25]

Description was changed from

==========
Clarify that Blink does not support re-distribution across v0 and v1 shadow
trees

BUG=531990
==========

to

==========
Clarify that Blink does not support a re-distribution across v0 and v1 shadow
trees

I do not have a plan to support a re-distribution across v0 and v1 shadow trees.
It would be better to clarify it in the comment.

BUG=531990
==========

[hayato, 2016-04-11 09:34:40]

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

[hayato, 2016-04-11 09:34:40]

PTAL

https://codereview.chromium.org/1877813002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/dom/shadow/SlotAssignment.cpp (right):

https://codereview.chromium.org/1877813002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/dom/shadow/SlotAssignment.cpp:43: if
(child.isInsertionPoint()) {
Strictly speaking, this is the behavior change. However, let me omit to write a
test for that because it is no worth having a test for that.

[kochi, 2016-04-11 10:35:34]

lgtm

https://codereview.chromium.org/1877813002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/core/dom/shadow/SlotAssignment.cpp (right):

https://codereview.chromium.org/1877813002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/core/dom/shadow/SlotAssignment.cpp:43: if
(child.isInsertionPoint()) {
On 2016/04/11 09:34:40, hayato wrote:
> Strictly speaking, this is the behavior change. However, let me omit to write
a
> test for that because it is no worth having a test for that.

Acknowledged.

[commit-bot: I haz the power, 2016-04-11 10:52:33]

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

[commit-bot: I haz the power, 2016-04-11 10:52:34]

Dry run: This issue passed the CQ dry run.

[hayato, 2016-04-12 07:16:13]

The CQ bit was checked by hayato@chromium.org

[commit-bot: I haz the power, 2016-04-12 07:16:25]

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

[commit-bot: I haz the power, 2016-04-12 07:21:17]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-12 07:22:59]

Description was changed from

==========
Clarify that Blink does not support a re-distribution across v0 and v1 shadow
trees

I do not have a plan to support a re-distribution across v0 and v1 shadow trees.
It would be better to clarify it in the comment.

BUG=531990
==========

to

==========
Clarify that Blink does not support a re-distribution across v0 and v1 shadow
trees

I do not have a plan to support a re-distribution across v0 and v1 shadow trees.
It would be better to clarify it in the comment.

BUG=531990

Committed: https://crrev.com/42739d8adae310a94b87f153878a0f1d4365618b
Cr-Commit-Position: refs/heads/master@{#386606}
==========

[commit-bot: I haz the power, 2016-04-12 07:23:01]

Patchset 2 (id:??) landed as
https://crrev.com/42739d8adae310a94b87f153878a0f1d4365618b
Cr-Commit-Position: refs/heads/master@{#386606}