Make assert_selection() to support no BODY element case

Make assert_selection() to support no BODY element result

This patch makes |assert_selection()| to serialize document element when
result document doesn't have a BODY element.

This patch is a preparation of http://crrev.com/2131113002, which makes
"justyFull" Command to work without BODY element.

BUG=n/a
TEST=LayoutTests/editing/assert_selection.html

Committed: https://crrev.com/8a079b530b47ce80a798f1dac8a3736c51a30558
Cr-Commit-Position: refs/heads/master@{#406503}

[yosin_UTC9, 2016-07-20 03:36:19]

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

[commit-bot: I haz the power, 2016-07-20 03:36:53]

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-07-20 03:38:37]

Description was changed from

==========
2016-07-20T12:34:05

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

to

==========
Make assert_selection() to support no BODY element result

This patch makes |assert_selection()| to serialize document element when
result document doesn't have a BODY element.

BUG=n/a
TEST=LayoutTests/editing/assert_selection.html
==========

[yosin_UTC9, 2016-07-20 03:40:38]

Description was changed from

==========
Make assert_selection() to support no BODY element result

This patch makes |assert_selection()| to serialize document element when
result document doesn't have a BODY element.

BUG=n/a
TEST=LayoutTests/editing/assert_selection.html
==========

to

==========
Make assert_selection() to support no BODY element result

This patch makes |assert_selection()| to serialize document element when
result document doesn't have a BODY element.

This patch is a preparation of http://crrev.com/2131113002, which makes
"justyFull" Command to work without BODY element.

BUG=n/a
TEST=LayoutTests/editing/assert_selection.html
==========

[yosin_UTC9, 2016-07-20 03:40:46]

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

[yosin_UTC9, 2016-07-20 03:40:46]

PTAL

[yoichio, 2016-07-20 04:20:30]

If your goal is to confirm not crashing,
 how about to introduce assert_selection_not_crash(inputText,tester,description)
 so that we don't need to care output?

https://codereview.chromium.org/2162063004/diff/20001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/editing/assert_selection.html (right):

https://codereview.chromium.org/2162063004/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/editing/assert_selection.html:72: 'Serialize
document element instead of document.body when is is null.');
"when 'it' is null." ?

[yosin_UTC9, 2016-07-20 04:37:18]

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

[commit-bot: I haz the power, 2016-07-20 04:37:35]

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-07-20 04:38:41]

On 2016/07/20 at 04:20:30, yoichio wrote:
> If your goal is to confirm not crashing,
>  how about to introduce
assert_selection_not_crash(inputText,tester,description)
>  so that we don't need to care output?

We should check result on crash test to avoid regression.
For crash test, it is better to use gTest when we can narrow down to the
function.

[yosin_UTC9, 2016-07-20 04:39:06]

PTAL

https://codereview.chromium.org/2162063004/diff/20001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/editing/assert_selection.html (right):

https://codereview.chromium.org/2162063004/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/editing/assert_selection.html:72: 'Serialize
document element instead of document.body when is is null.');
On 2016/07/20 at 04:20:30, yoichio wrote:
> "when 'it' is null." ?

Done

[yoichio, 2016-07-20 04:45:42]

lgtm

[yosin_UTC9, 2016-07-20 05:02:11]

The CQ bit was unchecked by yosin@chromium.org

[yosin_UTC9, 2016-07-20 05:02:13]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2016-07-20 05:02:17]

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-07-20 06:05:33]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-07-20 06:05:35]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-20 06:07:14]

Description was changed from

==========
Make assert_selection() to support no BODY element result

This patch makes |assert_selection()| to serialize document element when
result document doesn't have a BODY element.

This patch is a preparation of http://crrev.com/2131113002, which makes
"justyFull" Command to work without BODY element.

BUG=n/a
TEST=LayoutTests/editing/assert_selection.html
==========

to

==========
Make assert_selection() to support no BODY element result

This patch makes |assert_selection()| to serialize document element when
result document doesn't have a BODY element.

This patch is a preparation of http://crrev.com/2131113002, which makes
"justyFull" Command to work without BODY element.

BUG=n/a
TEST=LayoutTests/editing/assert_selection.html

Committed: https://crrev.com/8a079b530b47ce80a798f1dac8a3736c51a30558
Cr-Commit-Position: refs/heads/master@{#406503}
==========

[commit-bot: I haz the power, 2016-07-20 06:07:15]

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