Convert editing/deleting/delete-block-merge-contents-0{18,19,20,21}.html to use w3c test harness

Convert editing/deleting/delete-block-merge-contents-0{18,19,20,21}.html to use w3c test harness

This patch converts "editing/deleting/delete-block-merge-contents-0{18,19,20,21}.html" to
use w3c test harness to simplify test script for improving readability, and
consolidates into "delete_block_merge_end_of_line.html" since samples of them
are similar.

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

Committed: https://crrev.com/fe6da08d78aa26381effc626b952023a99a5da93
Cr-Commit-Position: refs/heads/master@{#412794}

[yosin_UTC9, 2016-08-18 05:47:31]

Description was changed from

==========
2016-08-18T14:44:04

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

to

==========
Convert editing/deleting/delete-block-merge-contents-0{18,19,20,21}.html to use
w3c test harness

This patch converts
"editing/deleting/delete-block-merge-contents-0{18,19,20,21}.html" to
use w3c test harness to simplify test script for improving readability, and
consolidates into "delete_block_merge_end_of_line.html" since samples of them
are similar.

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

[yosin_UTC9, 2016-08-18 05:47:38]

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

[commit-bot: I haz the power, 2016-08-18 05:47:47]

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-18 07:09:54]

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

[yosin_UTC9, 2016-08-18 07:09:55]

PTAL

[commit-bot: I haz the power, 2016-08-18 07:24:42]

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

[commit-bot: I haz the power, 2016-08-18 07:24:43]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Xiaocheng, 2016-08-18 07:55:16]

Only three test cases for four test files?

[yosin_UTC9, 2016-08-18 08:30:09]

On 2016/08/18 at 07:55:16, xiaochengh wrote:
> Only three test cases for four test files?

PTAL

Oops, I forgot to add a test case for delete-block-merge-contents-020.html

[Xiaocheng, 2016-08-18 09:01:41]

https://codereview.chromium.org/2254933003/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/editing/deleting/delete_block_merge_end_of_line.html
(right):

https://codereview.chromium.org/2254933003/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/editing/deleting/delete_block_merge_end_of_line.html:22:
test(() => assert_selection(
This doesn't seem to be the same test as in ...-019.html

I think it should be '<div>X^<div>should not see this content<br>|Y</div></div>'
=> '<div>X|Y</div>'

https://codereview.chromium.org/2254933003/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/editing/deleting/delete_block_merge_end_of_line.html:54:
test(() => assert_selection(
The original test seems to be '<div>X^<div>should not see this
content<br>Y|Z<br>AB</div></div>' => '<div>X|Z<div>AB</div></div>' instead.

[yosin_UTC9, 2016-08-18 09:21:16]

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

[commit-bot: I haz the power, 2016-08-18 09:21:31]

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-18 09:21:34]

PTAL

https://codereview.chromium.org/2254933003/diff/20001/third_party/WebKit/Layo...
File
third_party/WebKit/LayoutTests/editing/deleting/delete_block_merge_end_of_line.html
(right):

https://codereview.chromium.org/2254933003/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/editing/deleting/delete_block_merge_end_of_line.html:22:
test(() => assert_selection(
On 2016/08/18 at 09:01:40, Xiaocheng wrote:
> This doesn't seem to be the same test as in ...-019.html
> 
> I think it should be '<div>X^<div>should not see this
content<br>|Y</div></div>' => '<div>X|Y</div>'

Done. Thanks for catching this!

https://codereview.chromium.org/2254933003/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/editing/deleting/delete_block_merge_end_of_line.html:54:
test(() => assert_selection(
On 2016/08/18 at 09:01:40, Xiaocheng wrote:
> The original test seems to be '<div>X^<div>should not see this
content<br>Y|Z<br>AB</div></div>' => '<div>X|Z<div>AB</div></div>' instead.

Done. Thanks for catching this!

[Xiaocheng, 2016-08-18 09:25:08]

lgtm

[yosin_UTC9, 2016-08-18 09:48:04]

The CQ bit was unchecked by yosin@chromium.org

[yosin_UTC9, 2016-08-18 09:48:06]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2016-08-18 09:48:25]

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-18 10:49:18]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-18 10:51:09]

Description was changed from

==========
Convert editing/deleting/delete-block-merge-contents-0{18,19,20,21}.html to use
w3c test harness

This patch converts
"editing/deleting/delete-block-merge-contents-0{18,19,20,21}.html" to
use w3c test harness to simplify test script for improving readability, and
consolidates into "delete_block_merge_end_of_line.html" since samples of them
are similar.

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

to

==========
Convert editing/deleting/delete-block-merge-contents-0{18,19,20,21}.html to use
w3c test harness

This patch converts
"editing/deleting/delete-block-merge-contents-0{18,19,20,21}.html" to
use w3c test harness to simplify test script for improving readability, and
consolidates into "delete_block_merge_end_of_line.html" since samples of them
are similar.

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

Committed: https://crrev.com/fe6da08d78aa26381effc626b952023a99a5da93
Cr-Commit-Position: refs/heads/master@{#412794}
==========

[commit-bot: I haz the power, 2016-08-18 10:51:10]

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