Convert editing/undo/4063751.html to use w3c test harness

Convert editing/undo/4063751.html to use w3c test harness

This patch converts "editing/undo/4063751.html" to use w3c test harness
to simplify test script for improving readability.

This patch also renames it to "redo_rebalance_whitespace.html" to provide a
hint what this test script verifies.

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

Committed: https://crrev.com/14da44c0d317243b314d97b04b57b47c9888f127
Cr-Commit-Position: refs/heads/master@{#412749}

[yosin_UTC9, 2016-08-16 06:25:18]

Description was changed from

==========
2016-08-16T15:22:16

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

to

==========
Convert editing/undo/4063751.html to use w3c test harness

This patch converts "editing/undo/4063751.html" to use w3c test harness
to simplify test script for improving readability.

This patch also renames it to "redo_rebalance_whitespace.html" to provide a
hint what this test script verifies.

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

[yosin_UTC9, 2016-08-16 06:25:23]

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

[commit-bot: I haz the power, 2016-08-16 06:25:40]

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-16 07:23:47]

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

[yosin_UTC9, 2016-08-16 07:23:47]

PTAL

[commit-bot: I haz the power, 2016-08-16 07:36:25]

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

[commit-bot: I haz the power, 2016-08-16 07:36:25]

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

[Xiaocheng, 2016-08-17 01:45:47]

Please remove redo-style.html from this patch. It's already converted, iirc.

[yosin_UTC9, 2016-08-17 02:03:31]

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

[commit-bot: I haz the power, 2016-08-17 02:03:48]

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-17 02:03:49]

On 2016/08/17 at 01:45:47, xiaochengh wrote:
> Please remove redo-style.html from this patch. It's already converted, iirc.

PTAL

Oops, removed files.

[commit-bot: I haz the power, 2016-08-17 02:10:44]

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

[commit-bot: I haz the power, 2016-08-17 02:10:45]

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

[Xiaocheng, 2016-08-17 02:32:33]

On 2016/08/17 at 02:03:49, yosin wrote:
> On 2016/08/17 at 01:45:47, xiaochengh wrote:
> > Please remove redo-style.html from this patch. It's already converted, iirc.
> 
> PTAL
> 
> Oops, removed files.

Could you fix the presubmit error, which looks a little bit weird though?

[yosin_UTC9, 2016-08-17 04:36:18]

On 2016/08/17 at 02:32:33, xiaochengh wrote:
> On 2016/08/17 at 02:03:49, yosin wrote:
> > On 2016/08/17 at 01:45:47, xiaochengh wrote:
> > > Please remove redo-style.html from this patch. It's already converted,
iirc.
> > 
> > PTAL
> > 
> > Oops, removed files.
> 
> Could you fix the presubmit error, which looks a little bit weird though?

It seems bot config error(?).

The error is
OSError: [Errno 2] No such file or directory:
'/b/build/slave/linux/build/src/third_party/WebKit/LayoutTests/platform/linux/editing/undo'
step returned non-zero exit code: 1

4063751-expected.png and 4063751-expected.txt are last files in
"third_party/WebKit/LayoutTests/platform/linux/editing/undo".

[yosin_UTC9, 2016-08-17 04:36:44]

Description was changed from

==========
Convert editing/undo/4063751.html to use w3c test harness

This patch converts "editing/undo/4063751.html" to use w3c test harness
to simplify test script for improving readability.

This patch also renames it to "redo_rebalance_whitespace.html" to provide a
hint what this test script verifies.

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

to

==========
Convert editing/undo/4063751.html to use w3c test harness

This patch converts "editing/undo/4063751.html" to use w3c test harness
to simplify test script for improving readability.

This patch also renames it to "redo_rebalance_whitespace.html" to provide a
hint what this test script verifies.

BUG=n/a
TEST=n/a; no behavior changes
TBR=xiaochengh@chromium.org, yosin@chromium.org
==========

[yosin_UTC9, 2016-08-17 04:36:47]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2016-08-17 04:36:59]

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

[yosin_UTC9, 2016-08-17 04:37:33]

On 2016/08/17 at 04:36:18, Yosi_UTC9 wrote:
> On 2016/08/17 at 02:32:33, xiaochengh wrote:
> > On 2016/08/17 at 02:03:49, yosin wrote:
> > > On 2016/08/17 at 01:45:47, xiaochengh wrote:
> > > > Please remove redo-style.html from this patch. It's already converted,
iirc.
> > > 
> > > PTAL
> > > 
> > > Oops, removed files.
> > 
> > Could you fix the presubmit error, which looks a little bit weird though?
> 
> It seems bot config error(?).
> 
> The error is
> OSError: [Errno 2] No such file or directory:
'/b/build/slave/linux/build/src/third_party/WebKit/LayoutTests/platform/linux/editing/undo'
> step returned non-zero exit code: 1
> 
> 4063751-expected.png and 4063751-expected.txt are last files in
"third_party/WebKit/LayoutTests/platform/linux/editing/undo".

Attempt to "real" commit to see what will happen.

[commit-bot: I haz the power, 2016-08-17 04:40:10]

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

[commit-bot: I haz the power, 2016-08-17 04:40:11]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[yosin_UTC9, 2016-08-17 04:48:51]

On 2016/08/17 at 04:37:33, Yosi_UTC9 wrote:
> On 2016/08/17 at 04:36:18, Yosi_UTC9 wrote:
> > On 2016/08/17 at 02:32:33, xiaochengh wrote:
> > > On 2016/08/17 at 02:03:49, yosin wrote:
> > > > On 2016/08/17 at 01:45:47, xiaochengh wrote:
> > > > > Please remove redo-style.html from this patch. It's already converted,
iirc.
> > > > 
> > > > PTAL
> > > > 
> > > > Oops, removed files.
> > > 
> > > Could you fix the presubmit error, which looks a little bit weird though?
> > 
> > It seems bot config error(?).
> > 
> > The error is
> > OSError: [Errno 2] No such file or directory:
'/b/build/slave/linux/build/src/third_party/WebKit/LayoutTests/platform/linux/editing/undo'
> > step returned non-zero exit code: 1
> > 
> > 4063751-expected.png and 4063751-expected.txt are last files in
"third_party/WebKit/LayoutTests/platform/linux/editing/undo".
> 
> Attempt to "real" commit to see what will happen.

Filed: http://crbug.com/638478 Can't commit a patch making empty directory

[yosin_UTC9, 2016-08-18 02:11:34]

PTAL; presubmit bug is resolved.

[yosin_UTC9, 2016-08-18 02:11:41]

Description was changed from

==========
Convert editing/undo/4063751.html to use w3c test harness

This patch converts "editing/undo/4063751.html" to use w3c test harness
to simplify test script for improving readability.

This patch also renames it to "redo_rebalance_whitespace.html" to provide a
hint what this test script verifies.

BUG=n/a
TEST=n/a; no behavior changes
TBR=xiaochengh@chromium.org, yosin@chromium.org
==========

to

==========
Convert editing/undo/4063751.html to use w3c test harness

This patch converts "editing/undo/4063751.html" to use w3c test harness
to simplify test script for improving readability.

This patch also renames it to "redo_rebalance_whitespace.html" to provide a
hint what this test script verifies.

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

[Xiaocheng, 2016-08-18 02:13:20]

The CQ bit was checked by xiaochengh@chromium.org

[Xiaocheng, 2016-08-18 02:13:20]

lgtm

[commit-bot: I haz the power, 2016-08-18 02:13:49]

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 05:35:51]

Committed patchset #2 (id:20001)

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

Description was changed from

==========
Convert editing/undo/4063751.html to use w3c test harness

This patch converts "editing/undo/4063751.html" to use w3c test harness
to simplify test script for improving readability.

This patch also renames it to "redo_rebalance_whitespace.html" to provide a
hint what this test script verifies.

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

to

==========
Convert editing/undo/4063751.html to use w3c test harness

This patch converts "editing/undo/4063751.html" to use w3c test harness
to simplify test script for improving readability.

This patch also renames it to "redo_rebalance_whitespace.html" to provide a
hint what this test script verifies.

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

Committed: https://crrev.com/14da44c0d317243b314d97b04b57b47c9888f127
Cr-Commit-Position: refs/heads/master@{#412749}
==========

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

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