Convert editing/style/smoosh-styles-00[13].html to use w3c test harness

Convert editing/style/smoosh-styles-00[13].html to use w3c test harness

This patch converts "editing/style/smoosh-styles-00[13].html"
to use w3c test harness to simplify test script for improving readability, and
consolidates into "smoosh_styles.html" since these test scripts verifies
"smoosh style".

BUG=n/a
TEST=n/a; no behavior changes

Committed: https://crrev.com/6726ad114b333240bb5bf0de98ce7b9b3df6b971
Cr-Commit-Position: refs/heads/master@{#416231}

[yosin_UTC9, 2016-08-31 01:49:41]

Description was changed from

==========
2016-08-31T10:41:57

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

to

==========
Convert editing/style/smoosh-style-00[13].html to use w3c test harness

This patch converts "editing/style/smoosh-style-00[13].html"
to use w3c test harness to simplify test script for improving readability, and
consolidates into "smoosh_style.html" since these test scripts verifies
"smoosh style".

BUG=n/a
TEST=n/a; no behavior changes
==========

[yosin_UTC9, 2016-08-31 01:50:08]

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

[commit-bot: I haz the power, 2016-08-31 01:50:34]

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

[yosin_UTC9, 2016-08-31 01:50:59]

Description was changed from

==========
Convert editing/style/smoosh-style-00[13].html to use w3c test harness

This patch converts "editing/style/smoosh-style-00[13].html"
to use w3c test harness to simplify test script for improving readability, and
consolidates into "smoosh_style.html" since these test scripts verifies
"smoosh style".

BUG=n/a
TEST=n/a; no behavior changes
==========

to

==========
Convert editing/style/smoosh-styles-00[13].html to use w3c test harness

This patch converts "editing/style/smoosh-styles-00[13].html"
to use w3c test harness to simplify test script for improving readability, and
consolidates into "smoosh_styles.html" since these test scripts verifies
"smoosh style".

BUG=n/a
TEST=n/a; no behavior changes
==========

[commit-bot: I haz the power, 2016-08-31 03:48:09]

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

[commit-bot: I haz the power, 2016-08-31 03:48:11]

Dry run: This issue passed the CQ dry run.

[yosin_UTC9, 2016-08-31 04:17:11]

yosin@chromium.org changed reviewers:
+ xiaochengh@chromium.org, yoichio@chromium.org

[yosin_UTC9, 2016-08-31 04:17:12]

PTAL

[yoichio, 2016-08-31 08:20:25]

https://codereview.chromium.org/2294193002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/editing/style/smoosh_styles.html (right):

https://codereview.chromium.org/2294193002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/editing/style/smoosh_styles.html:20:
selection.modify('move', 'forward', 'character');
Could you use collapse()?

[yosin_UTC9, 2016-08-31 09:58:29]

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

[yosin_UTC9, 2016-08-31 09:58:40]

PTAL

https://codereview.chromium.org/2294193002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/editing/style/smoosh_styles.html (right):

https://codereview.chromium.org/2294193002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/editing/style/smoosh_styles.html:20:
selection.modify('move', 'forward', 'character');
On 2016/08/31 at 08:20:25, yoichio wrote:
> Could you use collapse()?

Done.

[commit-bot: I haz the power, 2016-08-31 09:58:51]

Dry run: 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-08-31 11:14:38]

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

[commit-bot: I haz the power, 2016-08-31 11:14:39]

Dry run: This issue passed the CQ dry run.

[yoichio, 2016-09-01 07:56:46]

https://codereview.chromium.org/2294193002/diff/20001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/editing/style/smoosh_styles.html (right):

https://codereview.chromium.org/2294193002/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/editing/style/smoosh_styles.html:20:
selection.document.execCommand('paste');
Can we have clipboard function in assert_selection to simplify cases?
It may look like:
assert_selection(
 '<div contenteditable>foo|bar</div>',
 selection => {
   selection.setClipboardHTML('<div>baz</div>');
   selection.document.execCommand('paste');
 },
 '<div contenteditable>foo<div>baz|</div>bar</div>');

In addition, if we have selection.getClipboardHTML, we also
 can confirm only what copy/cut command does:
assert_selection(
 '<div contenteditable>^foo|bar</div>',
 selection => {
   selection.document.execCommand('cut');
   assert_equals('foo', selection.clipboardHTML);
 },
 '<div contenteditable>|bar</div>');

[yosin_UTC9, 2016-09-02 05:14:45]

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

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

Dry run: 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:19:10]

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

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

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_compile_dbg_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_clobber_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[yosin_UTC9, 2016-09-02 07:29:36]

PTAL

Using selection.setClipboardData()

[yoichio, 2016-09-02 08:17:55]

lgtm

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

The CQ bit was checked by yosin@chromium.org

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

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 11:00:06]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-02 11:02:23]

Description was changed from

==========
Convert editing/style/smoosh-styles-00[13].html to use w3c test harness

This patch converts "editing/style/smoosh-styles-00[13].html"
to use w3c test harness to simplify test script for improving readability, and
consolidates into "smoosh_styles.html" since these test scripts verifies
"smoosh style".

BUG=n/a
TEST=n/a; no behavior changes
==========

to

==========
Convert editing/style/smoosh-styles-00[13].html to use w3c test harness

This patch converts "editing/style/smoosh-styles-00[13].html"
to use w3c test harness to simplify test script for improving readability, and
consolidates into "smoosh_styles.html" since these test scripts verifies
"smoosh style".

BUG=n/a
TEST=n/a; no behavior changes

Committed: https://crrev.com/6726ad114b333240bb5bf0de98ce7b9b3df6b971
Cr-Commit-Position: refs/heads/master@{#416231}
==========

[commit-bot: I haz the power, 2016-09-02 11:02:24]

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