Don't expose UA shadow roots in custom element callbacks

Don't expose UA shadow roots in custom element callbacks

SVG <use> can be used to gain access to a UA ShadowRoot, from there you could
then appendChild an input and querySelector across the boundary, or alter the
internals of the ShadowRoot during the SVG <use> cloning process.

This patch introduces a check in the callback queue that makes us drop all
callbacks for elements in UA shadows.

Committed: https://crrev.com/1c318713def9834c409c0a59c42c86d3a525700e
Cr-Commit-Position: refs/heads/master@{#357033}

[esprehn, 2015-10-29 21:02:41]

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

[esprehn, 2015-10-29 21:04:25]

Description was changed from

==========
use shadow CE callbacks.

BUG=
==========

to

==========
Don't expose UA shadow roots in custom element callbacks

SVG <use> can be used to gain access to a UA ShadowRoot, from there you could
then appendChild an input and querySelector across the boundary, or alter the
internals of the ShadowRoot during the SVG <use> cloning process.

This patch introduces a check in the callback queue that makes us drop all
callbacks for elements in UA shadows.
==========

[esprehn, 2015-10-29 21:05:40]

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

[commit-bot: I haz the power, 2015-10-29 21:06:06]

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

[esprehn, 2015-10-29 22:15:37]

esprehn@chromium.org changed reviewers:
+ dglazkov@chromium.org, dominicc@chromium.org, hayato@chromium.org

[commit-bot: I haz the power, 2015-10-29 23:22:22]

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

[commit-bot: I haz the power, 2015-10-29 23:22:23]

Dry run: This issue passed the CQ dry run.

[dominicc (has gone to gerrit), 2015-10-30 02:42:39]

The CQ bit was checked by dominicc@chromium.org

[dominicc (has gone to gerrit), 2015-10-30 02:42:40]

lgtm

Nice, thank you.

https://codereview.chromium.org/1427783004/diff/40001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/fast/dom/custom/svg-use-shadow-tree.html
(right):

https://codereview.chromium.org/1427783004/diff/40001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/fast/dom/custom/svg-use-shadow-tree.html:29:
class ElementType extends superClass {
This nested class definition w/ superClass is very neat.

[commit-bot: I haz the power, 2015-10-30 02:43:15]

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

[commit-bot: I haz the power, 2015-10-30 02:47:36]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-10-30 02:48:38]

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