Remove the support of multiple shadow roots with a user agent shadow root

Remove the support of multiple shadow roots with a user agent shadow root

"Intent to Remove" is here: https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/VKQk3uQuJLQ

This CL removes the web-facing API, as a first step.
The other removals, such as a cleanup of each element, will be done separately in other CLs.

BUG=489947
Committed: https://crrev.com/3a686f1ed8e775db80511c61672f63685361387f
Cr-Commit-Position: refs/heads/master@{#378921}

[hayato, 2016-03-02 06:23:34]

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

[commit-bot: I haz the power, 2016-03-02 06:23:39]

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

[hayato, 2016-03-02 06:25:03]

Description was changed from

==========
Remove the support of multiple shadow roots with a user agent shadow root

"Intent to Remove" is here:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/VKQk3uQuJLQ

This CL removes the web-facing API, as a first step.
The other clean up tasks for each elements will be done separately in another
CL.

BUG=489947
==========

to

==========
Remove the support of multiple shadow roots with a user agent shadow root

"Intent to Remove" is here:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/VKQk3uQuJLQ

This CL removes the web-facing API, as a first step.
The other cleanup  for each elements will be done separately in other CLs.

BUG=489947
==========

[hayato, 2016-03-02 06:27:23]

Description was changed from

==========
Remove the support of multiple shadow roots with a user agent shadow root

"Intent to Remove" is here:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/VKQk3uQuJLQ

This CL removes the web-facing API, as a first step.
The other cleanup  for each elements will be done separately in other CLs.

BUG=489947
==========

to

==========
Remove the support of multiple shadow roots with a user agent shadow root

"Intent to Remove" is here:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/VKQk3uQuJLQ

This CL removes the web-facing API, as a first step.
The other removal, such as cleanup of each elements, will be done separately in
other CLs.

BUG=489947
==========

[hayato, 2016-03-02 06:27:49]

Description was changed from

==========
Remove the support of multiple shadow roots with a user agent shadow root

"Intent to Remove" is here:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/VKQk3uQuJLQ

This CL removes the web-facing API, as a first step.
The other removal, such as cleanup of each elements, will be done separately in
other CLs.

BUG=489947
==========

to

==========
Remove the support of multiple shadow roots with a user agent shadow root

"Intent to Remove" is here:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/VKQk3uQuJLQ

This CL removes the web-facing API, as a first step.
The other removals, such as a cleanup of each element, will be done separately
in other CLs.

BUG=489947
==========

[hayato, 2016-03-02 06:44:37]

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

[hayato, 2016-03-02 06:44:38]

PTAL

[tkent, 2016-03-02 06:52:20]

https://codereview.chromium.org/1751953003/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/fast/dom/shadow/shadow-disable-expected.txt
(right):

https://codereview.chromium.org/1751953003/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/fast/dom/shadow/shadow-disable-expected.txt:24:
FAIL element.createShadowRoot() should throw an exception. Was [object
ShadowRoot].
Failing.   It's ok to remove a test for APPLET because we removed it.

[hayato, 2016-03-02 07:03:58]

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

[hayato, 2016-03-02 07:04:02]

remove an applet test

[commit-bot: I haz the power, 2016-03-02 07:04:10]

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

[hayato, 2016-03-02 07:04:50]

https://codereview.chromium.org/1751953003/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/fast/dom/shadow/shadow-disable-expected.txt
(right):

https://codereview.chromium.org/1751953003/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/fast/dom/shadow/shadow-disable-expected.txt:24:
FAIL element.createShadowRoot() should throw an exception. Was [object
ShadowRoot].
On 2016/03/02 06:52:20, tkent wrote:
> Failing.   It's ok to remove a test for APPLET because we removed it.

Done.

[tkent, 2016-03-02 07:06:30]

lgtm

[hayato, 2016-03-02 07:06:46]

The CQ bit was unchecked by hayato@chromium.org

[hayato, 2016-03-02 07:06:46]

The CQ bit was checked by hayato@chromium.org

[commit-bot: I haz the power, 2016-03-02 07:06:52]

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

[commit-bot: I haz the power, 2016-03-02 08:29:48]

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

[commit-bot: I haz the power, 2016-03-02 08:29:49]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[hayato, 2016-03-02 08:31:07]

Looks the following tests are failing. Let me fix these.

svg/dom/custom-elements.html
fast/selectors/shadow-host-div-with-text.html
editing/spelling/spellcheck-author-shadow-crash.html
imported/web-platform-tests/shadow-dom/untriaged/html-elements-and-their-shadow-trees/test-004.html

[hayato, 2016-03-02 09:01:24]

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

[hayato, 2016-03-02 09:01:33]

fix failing tests

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

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

[commit-bot: I haz the power, 2016-03-02 10:32:19]

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

[commit-bot: I haz the power, 2016-03-02 10:32:21]

Dry run: This issue passed the CQ dry run.

[hayato, 2016-03-03 02:14:23]

The CQ bit was checked by hayato@chromium.org

[hayato, 2016-03-03 02:14:23]

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

[commit-bot: I haz the power, 2016-03-03 02:14:44]

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

[commit-bot: I haz the power, 2016-03-03 02:22:16]

Description was changed from

==========
Remove the support of multiple shadow roots with a user agent shadow root

"Intent to Remove" is here:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/VKQk3uQuJLQ

This CL removes the web-facing API, as a first step.
The other removals, such as a cleanup of each element, will be done separately
in other CLs.

BUG=489947
==========

to

==========
Remove the support of multiple shadow roots with a user agent shadow root

"Intent to Remove" is here:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/VKQk3uQuJLQ

This CL removes the web-facing API, as a first step.
The other removals, such as a cleanup of each element, will be done separately
in other CLs.

BUG=489947
==========

[commit-bot: I haz the power, 2016-03-03 02:22:18]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-03 02:24:50]

Description was changed from

==========
Remove the support of multiple shadow roots with a user agent shadow root

"Intent to Remove" is here:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/VKQk3uQuJLQ

This CL removes the web-facing API, as a first step.
The other removals, such as a cleanup of each element, will be done separately
in other CLs.

BUG=489947
==========

to

==========
Remove the support of multiple shadow roots with a user agent shadow root

"Intent to Remove" is here:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/VKQk3uQuJLQ

This CL removes the web-facing API, as a first step.
The other removals, such as a cleanup of each element, will be done separately
in other CLs.

BUG=489947

Committed: https://crrev.com/3a686f1ed8e775db80511c61672f63685361387f
Cr-Commit-Position: refs/heads/master@{#378921}
==========

[commit-bot: I haz the power, 2016-03-03 02:24:52]

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

[kochi, 2016-03-03 02:37:52]

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

[kochi, 2016-03-03 02:37:53]

will HTMLInputElement::willAddFirstAuthorShadowRoot() and the like
be removed in future CLs?

[hayato, 2016-03-03 04:19:26]

On 2016/03/03 02:37:53, kochi wrote:
> will HTMLInputElement::willAddFirstAuthorShadowRoot() and the like
> be removed in future CLs?

We should do it as the description of this CL says.

[kochi, 2016-03-03 05:03:22]

On 2016/03/03 04:19:26, hayato wrote:
> On 2016/03/03 02:37:53, kochi wrote:
> > will HTMLInputElement::willAddFirstAuthorShadowRoot() and the like
> > be removed in future CLs?
> 
> We should do it as the description of this CL says.

SGTM