Range.surroundContents() should not check node types beforehand.

Range.surroundContents() should not check node types beforehand.

Our implementation checked childTypeAllowed() beforehand.  However, this is not
defined by the DOM standard, and caused some test failures. This CL removes it.

Also, this CL corrects the order of step 3 and step 4 because the above removal
regressed some passing tests, and the order correction recovers them.

BUG=700266
Review-Url: https://codereview.chromium.org/2745533004
Cr-Commit-Position: refs/heads/master@{#456017}
Committed: https://chromium.googlesource.com/chromium/src/+/caf3c3acbe69ac013e2aaafad5c8110bb66e435a

[tkent, 2017-03-10 04:58:40]

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

[commit-bot: I haz the power, 2017-03-10 04:59:02]

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, 2017-03-10 06:15:20]

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

[commit-bot: I haz the power, 2017-03-10 06:15:20]

Dry run: This issue passed the CQ dry run.

[tkent, 2017-03-10 06:17:46]

Description was changed from

==========
Range.surroundContents() should not check node types beforehand.

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

to

==========
Range.surroundContents() should not check node types beforehand.

BUG=700266
==========

[tkent, 2017-03-10 06:28:41]

Description was changed from

==========
Range.surroundContents() should not check node types beforehand.

BUG=700266
==========

to

==========
Range.surroundContents() should not check node types beforehand.

Our implementation checked childTypeAllowed() beforehand.  However, this is not
defined by the DOM standard, and caused some test failures. This CL removes it.

Also, this CL corrects the order of step 3 and step 4 because the above removal
regressed some passing tests, and the order correction recovers them.

BUG=700266
==========

[tkent, 2017-03-10 06:28:48]

Description was changed from

==========
Range.surroundContents() should not check node types beforehand.

Our implementation checked childTypeAllowed() beforehand.  However, this is not
defined by the DOM standard, and caused some test failures. This CL removes it.

Also, this CL corrects the order of step 3 and step 4 because the above removal
regressed some passing tests, and the order correction recovers them.

BUG=700266
==========

to

==========
Range.surroundContents() should not check node types beforehand.

Our implementation checked childTypeAllowed() beforehand.  However, this is not
defined by the DOM standard, and caused some test failures. This CL removes it.

Also, this CL corrects the order of step 3 and step 4 because the above removal
regressed some passing tests, and the order correction recovers them.

BUG=700266
==========

[tkent, 2017-03-10 06:29:21]

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

[tkent, 2017-03-10 06:29:22]

yosin@, would you review this please?
This fixes 24 tests.

[yosin_UTC9, 2017-03-10 07:23:40]

The CQ bit was checked by yosin@chromium.org

[yosin_UTC9, 2017-03-10 07:23:40]

lgtm

[commit-bot: I haz the power, 2017-03-10 07:23:48]

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, 2017-03-10 07:27:17]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489130620077230, "parent_rev":
"e8b77cc6ed024fb219d6de46378bf18eb05ae7f6", "commit_rev":
"caf3c3acbe69ac013e2aaafad5c8110bb66e435a"}

[commit-bot: I haz the power, 2017-03-10 07:27:58]

Description was changed from

==========
Range.surroundContents() should not check node types beforehand.

Our implementation checked childTypeAllowed() beforehand.  However, this is not
defined by the DOM standard, and caused some test failures. This CL removes it.

Also, this CL corrects the order of step 3 and step 4 because the above removal
regressed some passing tests, and the order correction recovers them.

BUG=700266
==========

to

==========
Range.surroundContents() should not check node types beforehand.

Our implementation checked childTypeAllowed() beforehand.  However, this is not
defined by the DOM standard, and caused some test failures. This CL removes it.

Also, this CL corrects the order of step 3 and step 4 because the above removal
regressed some passing tests, and the order correction recovers them.

BUG=700266

Review-Url: https://codereview.chromium.org/2745533004
Cr-Commit-Position: refs/heads/master@{#456017}
Committed:
https://chromium.googlesource.com/chromium/src/+/caf3c3acbe69ac013e2aaafad5c8...
==========

[commit-bot: I haz the power, 2017-03-10 07:27:59]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/caf3c3acbe69ac013e2aaafad5c8...