Make slotchange events bubble

Make slotchange events bubble

The spec is: https://dom.spec.whatwg.org/#notify-mutation-observers ; open this link, and find-in-page by "slotchange".
This CL is a bug fix. The current behavior is unintentional.

BUG=641140
Committed: https://crrev.com/db300b7b07375236467cfcc9aad82b8f8affa36b
Cr-Commit-Position: refs/heads/master@{#416197}

[hayato, 2016-09-02 02:43:07]

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

[commit-bot: I haz the power, 2016-09-02 02:43:21]

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

[hayato, 2016-09-02 02:44:41]

fix

[hayato, 2016-09-02 02:44:42]

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

[commit-bot: I haz the power, 2016-09-02 02:44:46]

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

[hayato, 2016-09-02 02:45:10]

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

[hayato, 2016-09-02 02:45:11]

PTAL

[tkent, 2016-09-02 02:53:15]

lgtm

[kochi, 2016-09-02 02:57:06]

lgtm

https://codereview.chromium.org/2306873002/diff/20001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/shadow-dom/slotchange.html (right):

https://codereview.chromium.org/2306873002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/shadow-dom/slotchange.html:33:
n.shadowroot.addEventListener('slotchange', test.step_func((e) => {
nit: you can use test.step_func_done(...) and remove line 37.

https://codereview.chromium.org/2306873002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/shadow-dom/slotchange.html:43: }, 'A slotchange
event shoud be a bubble, non-composed event.');
nit: s/bubble/bubbling/

[hayato, 2016-09-02 04:07:33]

fix

[hayato, 2016-09-02 04:07:34]

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

[commit-bot: I haz the power, 2016-09-02 04:07:58]

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

[hayato, 2016-09-02 04:08:10]

https://codereview.chromium.org/2306873002/diff/20001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/shadow-dom/slotchange.html (right):

https://codereview.chromium.org/2306873002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/shadow-dom/slotchange.html:33:
n.shadowroot.addEventListener('slotchange', test.step_func((e) => {
Done.

https://codereview.chromium.org/2306873002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/shadow-dom/slotchange.html:43: }, 'A slotchange
event shoud be a bubble, non-composed event.');
Done

[hayato, 2016-09-02 04:08:13]

The CQ bit was unchecked by hayato@chromium.org

[hayato, 2016-09-02 04:08:15]

The CQ bit was checked by hayato@chromium.org

[hayato, 2016-09-02 04:08:16]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org,
kochi@chromium.org
Link to the patchset: https://codereview.chromium.org/2306873002/#ps40001
(title: "fix")

[commit-bot: I haz the power, 2016-09-02 04:08:33]

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-09-02 05:47:40]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-02 05:49:47]

Description was changed from

==========
Make slotchange events bubble

The spec is: https://dom.spec.whatwg.org/#notify-mutation-observers ; open this
link, and find-in-page by "slotchange".
This CL is a bug fix. The current behavior is unintentional.

BUG=641140
==========

to

==========
Make slotchange events bubble

The spec is: https://dom.spec.whatwg.org/#notify-mutation-observers ; open this
link, and find-in-page by "slotchange".
This CL is a bug fix. The current behavior is unintentional.

BUG=641140

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

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

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