Convert editing/inserting/return-key-with-selection-00[123].html to use w3c test harness

Convert editing/inserting/return-key-with-selection-00[123].html to use w3c test harness

This patch converts "editing/inserting/return-key-with-selection-00[123].html"
to use w3c test harness to simplify test script for improving readability, and
consolidates into "return_key_with_selection.html" since these files use
similar test sample.

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

Committed: https://crrev.com/8557b5029b080915027dbdf409fb49b2d38bacfe
Cr-Commit-Position: refs/heads/master@{#415265}

[yosin_UTC9, 2016-08-30 06:45:39]

Description was changed from

==========
2016-08-30T15:40:09

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

to

==========
Convert editing/inserting/return-key-with-selection-00[123].html to use w3c test
harness

This patch converts "editing/inserting/return-key-with-selection-00[123].html"
to use w3c test harness to simplify test script for improving readability, and
consolidates into "return_key_with_selection.html" since these files use
similar test sample.

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

[yosin_UTC9, 2016-08-30 06:45:44]

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

[commit-bot: I haz the power, 2016-08-30 06:46:06]

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-30 07:37:05]

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

[yosin_UTC9, 2016-08-30 07:37:06]

PTAL

[Xiaocheng, 2016-08-30 08:04:45]

https://codereview.chromium.org/2293783002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/editing/inserting/return_key_with_selection.html
(right):

https://codereview.chromium.org/2293783002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/editing/inserting/return_key_with_selection.html:10:
'<div>foo</div><div>^bar</div>|<div>baz</div></div>',
There is an extra </div>.

And should the focus be like '...<div>|baz</div>'? Or does it matter?

https://codereview.chromium.org/2293783002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/editing/inserting/return_key_with_selection.html:38:
'<div contenteditable >',
There is an extra space.

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

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

[commit-bot: I haz the power, 2016-08-30 08:06:11]

Dry run: This issue passed the CQ dry run.

[yosin_UTC9, 2016-08-30 08:32:38]

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

[commit-bot: I haz the power, 2016-08-30 08:32:54]

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-30 08:33:04]

PTAL

https://codereview.chromium.org/2293783002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/editing/inserting/return_key_with_selection.html
(right):

https://codereview.chromium.org/2293783002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/editing/inserting/return_key_with_selection.html:10:
'<div>foo</div><div>^bar</div>|<div>baz</div></div>',
On 2016/08/30 at 08:04:45, Xiaocheng wrote:
> There is an extra </div>.
Removed.

> And should the focus be like '...<div>|baz</div>'? Or does it matter?
No. I added test case for "<div>|baz</div>" for record of current behavior.

https://codereview.chromium.org/2293783002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/editing/inserting/return_key_with_selection.html:38:
'<div contenteditable >',
On 2016/08/30 at 08:04:45, Xiaocheng wrote:
> There is an extra space.

Done.

[Xiaocheng, 2016-08-30 09:17:15]

lgtm

[commit-bot: I haz the power, 2016-08-30 09:21:48]

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

[commit-bot: I haz the power, 2016-08-30 09:21:49]

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

[yosin_UTC9, 2016-08-30 09:22:06]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2016-08-30 09:22:19]

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-30 10:33:35]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-30 10:35:43]

Description was changed from

==========
Convert editing/inserting/return-key-with-selection-00[123].html to use w3c test
harness

This patch converts "editing/inserting/return-key-with-selection-00[123].html"
to use w3c test harness to simplify test script for improving readability, and
consolidates into "return_key_with_selection.html" since these files use
similar test sample.

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

to

==========
Convert editing/inserting/return-key-with-selection-00[123].html to use w3c test
harness

This patch converts "editing/inserting/return-key-with-selection-00[123].html"
to use w3c test harness to simplify test script for improving readability, and
consolidates into "return_key_with_selection.html" since these files use
similar test sample.

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

Committed: https://crrev.com/8557b5029b080915027dbdf409fb49b2d38bacfe
Cr-Commit-Position: refs/heads/master@{#415265}
==========

[commit-bot: I haz the power, 2016-08-30 10:35:44]

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