ComposedTreeWalker does't have to return a node if an insertion point has no distributed nodes.

ComposedTreeWalker does't have to return a node if an insertion point has no distributed nodes.

A content insertion point never has child nodes if the insertion point has distributed nodes after r161137.

A shadow insertion point may have child nodes even if the insertion point has
no distributed nodes.  We should return 0 in this case because child nodes of the
shadow insertion point must not be treated as fallback elements after r161137.

The relevant spec is:
https://dvcs.w3.org/hg/webcomponents/raw-file/tip/spec/shadow/index.html#distributions

BUG=None

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=161596

[hayato, 2013-11-06 10:29:14]

PTAL

[Hajime Morrita, 2013-11-07 04:37:25]

lgtm.
If there is relevant section in the spec,
it would be great to have a link for that from the commit log.

[hayato, 2013-11-07 05:31:33]

On 2013/11/07 04:37:25, morrita1 wrote:
> lgtm.
> If there is relevant section in the spec,
> it would be great to have a link for that from the commit log.

Thank you.
https://dvcs.w3.org/hg/webcomponents/raw-file/tip/spec/shadow/index.html#dist...
is a good candidate for a link.

The description is the deduction of the result of running the algorithms.

[commit-bot: I haz the power, 2013-11-07 05:32:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/hayato@chromium.org/60203009/1

[commit-bot: I haz the power, 2013-11-07 06:03:24]

Sorry for I got bad news for ya.
Compile failed with a clobber build on linux_blink.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_blin...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-11-08 07:11:55]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/hayato@chromium.org/60203009/140001

[commit-bot: I haz the power, 2013-11-08 08:07:03]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/hayato@chromium.org/60203009/140001

[commit-bot: I haz the power, 2013-11-08 10:02:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/hayato@chromium.org/60203009/140001

[commit-bot: I haz the power, 2013-11-08 10:05:09]

Change committed as 161596