Rewrite tests for slots

Rewrite tests for slots

Make the test more comprehensive, organized, and readable.

BUG=531990
Committed: https://crrev.com/6502220de4e8cb4dbb28fe0ea21035453b02d3ac
Cr-Commit-Position: refs/heads/master@{#400152}

[hayato, 2016-06-13 09:17:48]

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

[commit-bot: I haz the power, 2016-06-13 09:18:03]

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

[commit-bot: I haz the power, 2016-06-13 09:19:55]

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

[commit-bot: I haz the power, 2016-06-13 09:19:56]

Dry run: Try jobs failed on following builders:
  ios-device on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-gn/bui...)
  ios-simulator on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-gn/...)

[hayato, 2016-06-13 09:33:18]

Description was changed from

==========
Rewrite tests for slots

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

to

==========
Rewrite tests for slots

Make the test more comprehensive, organized, and readable.

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

[hayato, 2016-06-13 09:33:38]

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

[tkent, 2016-06-15 13:23:22]

lgtm

https://codereview.chromium.org/2060793002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/shadow-dom/slots.html (right):

https://codereview.chromium.org/2060793002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/shadow-dom/slots.html:173: <div id="c2"
slot="slot1""></div>
nit: Remove a redundant double-quote.

[hayato, 2016-06-16 07:48:58]

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

[hayato, 2016-06-16 07:49:00]

fixed

[commit-bot: I haz the power, 2016-06-16 07:49:09]

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

[hayato, 2016-06-16 07:49:24]

Thank you for the review.

https://codereview.chromium.org/2060793002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/shadow-dom/slots.html (right):

https://codereview.chromium.org/2060793002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/shadow-dom/slots.html:173: <div id="c2"
slot="slot1""></div>
Done

[commit-bot: I haz the power, 2016-06-16 07:52:28]

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

[commit-bot: I haz the power, 2016-06-16 07:52:29]

Dry run: Try jobs failed on following builders:
  android_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  cast_shell_android on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  cast_shell_linux on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  chromeos_amd64-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)
  chromeos_x86-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_x86-ge...)
  linux_chromium_asan_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_compile_dbg_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_clobber_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_compile_dbg_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[hayato, 2016-06-16 09:34:22]

The CQ bit was checked by hayato@chromium.org

[hayato, 2016-06-16 09:34:22]

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/2060793002/#ps20001
(title: "fixed")

[commit-bot: I haz the power, 2016-06-16 09:34:47]

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

[commit-bot: I haz the power, 2016-06-16 09:34:48]

This CL has an open dependency (Issue 2053353002 Patch 40001). Please resolve
the dependency and try again.
If you are sure that there is no real dependency, please use one of the options
listed in https://goo.gl/9Es4OR to land the CL.

[hayato, 2016-06-16 10:48:09]

The CQ bit was checked by hayato@chromium.org

[commit-bot: I haz the power, 2016-06-16 10:48:30]

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

[commit-bot: I haz the power, 2016-06-16 14:33:20]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-06-16 14:33:30]

CQ bit was unchecked

[commit-bot: I haz the power, 2016-06-16 14:34:33]

Description was changed from

==========
Rewrite tests for slots

Make the test more comprehensive, organized, and readable.

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

to

==========
Rewrite tests for slots

Make the test more comprehensive, organized, and readable.

BUG=531990

Committed: https://crrev.com/6502220de4e8cb4dbb28fe0ea21035453b02d3ac
Cr-Commit-Position: refs/heads/master@{#400152}
==========

[commit-bot: I haz the power, 2016-06-16 14:34:34]

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