Fix HTML parser CDATA edge-case and sync state names with spec

Fix HTML parser CDATA edge-case and sync state names with spec

When encountering a sequence of ']]]' at the end of a CDATA section, we
should only buffer one ']' (the first one) and "remember" the other two.

The states exists in the spec[1][2][3] nowadays, so sync the names and remove
the comment about the states not being in the spec.

Fixes two subtests of wpt/html/syntax/parsing/html5lib_tests21.html.

[1] https://html.spec.whatwg.org/multipage/syntax.html#cdata-section-state
[2] https://html.spec.whatwg.org/multipage/syntax.html#cdata-section-bracket-state
[3] https://html.spec.whatwg.org/multipage/syntax.html#cdata-section-end-state

BUG=674496
Committed: https://crrev.com/e02441191c08939fd05e7a5fdf619b68e91558a2
Cr-Commit-Position: refs/heads/master@{#439396}

[fs, 2016-12-15 09:59:26]

The CQ bit was checked by fs@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-12-15 09:59:44]

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-12-15 11:21:53]

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

[commit-bot: I haz the power, 2016-12-15 11:21:53]

Dry run: This issue passed the CQ dry run.

[fs, 2016-12-15 12:14:47]

Description was changed from

==========
Fix HTML parser CDATA edge-case and sync state names with spec

When encountering a sequence of ']]]' at the end of a CDATA section, we
should only buffer one ']' (the first one) and "remember" the other two.

The states exists in the spec nowadays, so sync the names and remove the
comment about the states not being in the spec.

Fixes two subtests of wpt/html/syntax/parsing/html5lib_tests21.html.

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

to

==========
Fix HTML parser CDATA edge-case and sync state names with spec

When encountering a sequence of ']]]' at the end of a CDATA section, we
should only buffer one ']' (the first one) and "remember" the other two.

The states exists in the spec nowadays, so sync the names and remove the
comment about the states not being in the spec.

Fixes two subtests of wpt/html/syntax/parsing/html5lib_tests21.html.

BUG=674496
==========

[fs, 2016-12-15 12:16:22]

Description was changed from

==========
Fix HTML parser CDATA edge-case and sync state names with spec

When encountering a sequence of ']]]' at the end of a CDATA section, we
should only buffer one ']' (the first one) and "remember" the other two.

The states exists in the spec nowadays, so sync the names and remove the
comment about the states not being in the spec.

Fixes two subtests of wpt/html/syntax/parsing/html5lib_tests21.html.

BUG=674496
==========

to

==========
Fix HTML parser CDATA edge-case and sync state names with spec

When encountering a sequence of ']]]' at the end of a CDATA section, we
should only buffer one ']' (the first one) and "remember" the other two.

The states exists in the spec[1][2][3] nowadays, so sync the names and remove
the comment about the states not being in the spec.

Fixes two subtests of wpt/html/syntax/parsing/html5lib_tests21.html.

[1] https://html.spec.whatwg.org/multipage/syntax.html#cdata-section-state
[2]
https://html.spec.whatwg.org/multipage/syntax.html#cdata-section-bracket-state
[3] https://html.spec.whatwg.org/multipage/syntax.html#cdata-section-end-state

BUG=674496
==========

[fs, 2016-12-15 12:16:51]

fs@opera.com changed reviewers:
+ kouhei@chromium.org

[kouhei (in TOK), 2016-12-19 00:50:38]

lgtm

I really appreciate this change! Thanks!

[kouhei (in TOK), 2016-12-19 00:50:41]

The CQ bit was checked by kouhei@chromium.org

[commit-bot: I haz the power, 2016-12-19 00:50: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-12-19 03:11:13]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1482108641642000, "parent_rev":
"6bc087f1c80b4cda3ed7759761d969ad7b51b3bb", "commit_rev":
"8a65c5f06e92046124da9a20e83d452451e38c6d"}

[commit-bot: I haz the power, 2016-12-19 03:11:51]

Description was changed from

==========
Fix HTML parser CDATA edge-case and sync state names with spec

When encountering a sequence of ']]]' at the end of a CDATA section, we
should only buffer one ']' (the first one) and "remember" the other two.

The states exists in the spec[1][2][3] nowadays, so sync the names and remove
the comment about the states not being in the spec.

Fixes two subtests of wpt/html/syntax/parsing/html5lib_tests21.html.

[1] https://html.spec.whatwg.org/multipage/syntax.html#cdata-section-state
[2]
https://html.spec.whatwg.org/multipage/syntax.html#cdata-section-bracket-state
[3] https://html.spec.whatwg.org/multipage/syntax.html#cdata-section-end-state

BUG=674496
==========

to

==========
Fix HTML parser CDATA edge-case and sync state names with spec

When encountering a sequence of ']]]' at the end of a CDATA section, we
should only buffer one ']' (the first one) and "remember" the other two.

The states exists in the spec[1][2][3] nowadays, so sync the names and remove
the comment about the states not being in the spec.

Fixes two subtests of wpt/html/syntax/parsing/html5lib_tests21.html.

[1] https://html.spec.whatwg.org/multipage/syntax.html#cdata-section-state
[2]
https://html.spec.whatwg.org/multipage/syntax.html#cdata-section-bracket-state
[3] https://html.spec.whatwg.org/multipage/syntax.html#cdata-section-end-state

BUG=674496

Review-Url: https://codereview.chromium.org/2576373002
==========

[commit-bot: I haz the power, 2016-12-19 03:11:52]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-19 03:14:09]

Description was changed from

==========
Fix HTML parser CDATA edge-case and sync state names with spec

When encountering a sequence of ']]]' at the end of a CDATA section, we
should only buffer one ']' (the first one) and "remember" the other two.

The states exists in the spec[1][2][3] nowadays, so sync the names and remove
the comment about the states not being in the spec.

Fixes two subtests of wpt/html/syntax/parsing/html5lib_tests21.html.

[1] https://html.spec.whatwg.org/multipage/syntax.html#cdata-section-state
[2]
https://html.spec.whatwg.org/multipage/syntax.html#cdata-section-bracket-state
[3] https://html.spec.whatwg.org/multipage/syntax.html#cdata-section-end-state

BUG=674496

Review-Url: https://codereview.chromium.org/2576373002
==========

to

==========
Fix HTML parser CDATA edge-case and sync state names with spec

When encountering a sequence of ']]]' at the end of a CDATA section, we
should only buffer one ']' (the first one) and "remember" the other two.

The states exists in the spec[1][2][3] nowadays, so sync the names and remove
the comment about the states not being in the spec.

Fixes two subtests of wpt/html/syntax/parsing/html5lib_tests21.html.

[1] https://html.spec.whatwg.org/multipage/syntax.html#cdata-section-state
[2]
https://html.spec.whatwg.org/multipage/syntax.html#cdata-section-bracket-state
[3] https://html.spec.whatwg.org/multipage/syntax.html#cdata-section-end-state

BUG=674496

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

[commit-bot: I haz the power, 2016-12-19 03:14:10]

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