Convert editing/selection/4932260-[123].html to utilize w3c test harness

Convert editing/selection/4932260-[123].html to utilize w3c test harness

This patch converts "4932260-[123].html" to utilize w3c test harness with
|assert_selection()| to simplify test case and consolidate into
"move_around_image.html" since these test files were committed in same patch for
improving code health.

This patch is a preparation of http://crrev.com/1958093002

BUG=625533
TEST=n/a; no behavior chagnes

Committed: https://crrev.com/4426277e20a694987c4a324b385d07e79279ff2c
Cr-Commit-Position: refs/heads/master@{#428290}

[yosin_UTC9, 2016-10-26 07:51:27]

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

[commit-bot: I haz the power, 2016-10-26 07:51:38]

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-10-26 07:52:25]

Description was changed from

==========
2016-10-26T16:41:10

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

to

==========
Convert editing/selection/4932260-[123].html to utilize w3c test harness

This patch converts "4932260-[123].html" to utilize w3c test harness with
|assert_selection()| to simplify test case and consolidate into
"move_around_image.html" since these test files were committed in same patch for
improving code health.

This patch is a preparation of http://crrev.com/1958093002

BUG=625533
TEST=n/a; no behavior chagnes
==========

[yosin_UTC9, 2016-10-26 08:45:50]

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

[yosin_UTC9, 2016-10-26 08:45:51]

PTAL

[commit-bot: I haz the power, 2016-10-26 08:52:02]

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

[commit-bot: I haz the power, 2016-10-26 08:52:03]

Dry run: This issue passed the CQ dry run.

[Xiaocheng, 2016-10-26 09:31:04]

https://codereview.chromium.org/2446893006/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/editing/selection/modify_move/move_around_image.html
(right):

https://codereview.chromium.org/2446893006/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/editing/selection/modify_move/move_around_image.html:40:
selection => selection.modify('move', 'forward', 'sentence'),
Should be move forward by word twice.

https://codereview.chromium.org/2446893006/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/editing/selection/modify_move/move_around_image.html:54:
selection => selection.modify('move', 'backward', 'sentence'),
Should be move backward by word twice.

[yosin_UTC9, 2016-10-26 09:40:39]

https://codereview.chromium.org/2446893006/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/editing/selection/modify_move/move_around_image.html
(right):

https://codereview.chromium.org/2446893006/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/editing/selection/modify_move/move_around_image.html:40:
selection => selection.modify('move', 'forward', 'sentence'),
On 2016/10/26 at 09:31:04, Xiaocheng wrote:
> Should be move forward by word twice.

No need to move twice to verify 'move-forward-sentence'.

In other words, we should verify minimal unit of action rather than combination.

https://codereview.chromium.org/2446893006/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/editing/selection/modify_move/move_around_image.html:54:
selection => selection.modify('move', 'backward', 'sentence'),
On 2016/10/26 at 09:31:04, Xiaocheng wrote:
> Should be move backward by word twice.

No need to move twice to verify 'move-backward-sentence'.

[Xiaocheng, 2016-10-26 09:48:23]

On 2016/10/26 at 09:40:39, yosin wrote:
>
https://codereview.chromium.org/2446893006/diff/1/third_party/WebKit/LayoutTe...
> File
third_party/WebKit/LayoutTests/editing/selection/modify_move/move_around_image.html
(right):
> 
>
https://codereview.chromium.org/2446893006/diff/1/third_party/WebKit/LayoutTe...
>
third_party/WebKit/LayoutTests/editing/selection/modify_move/move_around_image.html:40:
selection => selection.modify('move', 'forward', 'sentence'),
> On 2016/10/26 at 09:31:04, Xiaocheng wrote:
> > Should be move forward by word twice.
> 
> No need to move twice to verify 'move-forward-sentence'.
> 
> In other words, we should verify minimal unit of action rather than
combination.
> 
>
https://codereview.chromium.org/2446893006/diff/1/third_party/WebKit/LayoutTe...
>
third_party/WebKit/LayoutTests/editing/selection/modify_move/move_around_image.html:54:
selection => selection.modify('move', 'backward', 'sentence'),
> On 2016/10/26 at 09:31:04, Xiaocheng wrote:
> > Should be move backward by word twice.
> 
> No need to move twice to verify 'move-backward-sentence'.

The original test is for a bug about word boundary, which is the reason that the
selection should be moved by word instead of sentence.

[yosin_UTC9, 2016-10-27 05:46:47]

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

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

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-10-27 06:46:48]

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

[commit-bot: I haz the power, 2016-10-27 06:46:49]

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_...)

[yosin_UTC9, 2016-10-27 07:51:22]

PTAL

Update to follow review comments.

[Xiaocheng, 2016-10-27 08:01:23]

lgtm

[yosin_UTC9, 2016-10-27 08:01:36]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2016-10-27 08:01:54]

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-10-27 09:23:45]

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

[commit-bot: I haz the power, 2016-10-27 09:23:46]

Try jobs failed on following builders:
  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-10-28 02:03:58]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2016-10-28 02:04:29]

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-10-28 03:25:41]

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

[commit-bot: I haz the power, 2016-10-28 03:25:42]

Try jobs failed on following builders:
  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-10-28 04:01:10]

The CQ bit was checked by yosin@chromium.org

[yosin_UTC9, 2016-10-28 04:01:11]

The patchset sent to the CQ was uploaded after l-g-t-m from
xiaochengh@chromium.org
Link to the patchset: https://codereview.chromium.org/2446893006/#ps40001
(title: "2016-10-28T12:57:04")

[commit-bot: I haz the power, 2016-10-28 04:01:24]

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-10-28 05:16:41]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-28 05:18:42]

Description was changed from

==========
Convert editing/selection/4932260-[123].html to utilize w3c test harness

This patch converts "4932260-[123].html" to utilize w3c test harness with
|assert_selection()| to simplify test case and consolidate into
"move_around_image.html" since these test files were committed in same patch for
improving code health.

This patch is a preparation of http://crrev.com/1958093002

BUG=625533
TEST=n/a; no behavior chagnes
==========

to

==========
Convert editing/selection/4932260-[123].html to utilize w3c test harness

This patch converts "4932260-[123].html" to utilize w3c test harness with
|assert_selection()| to simplify test case and consolidate into
"move_around_image.html" since these test files were committed in same patch for
improving code health.

This patch is a preparation of http://crrev.com/1958093002

BUG=625533
TEST=n/a; no behavior chagnes

Committed: https://crrev.com/4426277e20a694987c4a324b385d07e79279ff2c
Cr-Commit-Position: refs/heads/master@{#428290}
==========

[commit-bot: I haz the power, 2016-10-28 05:18:43]

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