[regexp] Fix incorrect DCHECK in FixSingleCharacterDisjunctions

[regexp] Fix incorrect DCHECK in FixSingleCharacterDisjunctions

The condition only applies in unicode mode, where any lone surrogates
are desugared into a character class (and will not be considered in this
optimization). Non-unicode mode treats lone surrogates exactly like
any other codepoint.

BUG=chromium:711092
Review-Url: https://codereview.chromium.org/2808403006
Cr-Commit-Position: refs/heads/master@{#44638}
Committed: https://chromium.googlesource.com/v8/v8/+/876e23c54d5fa2cb2a399632175d94e597da5330

[jgruber, 2017-04-13 09:17:06]

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

[commit-bot: I haz the power, 2017-04-13 09:17:12]

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

[jgruber, 2017-04-13 09:17:31]

jgruber@chromium.org changed reviewers:
+ yangguo@chromium.org

[commit-bot: I haz the power, 2017-04-13 09:46:16]

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

[commit-bot: I haz the power, 2017-04-13 09:46:17]

Dry run: This issue passed the CQ dry run.

[jgruber, 2017-04-13 09:50:41]

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

[commit-bot: I haz the power, 2017-04-13 09:50:46]

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-04-13 10:17:00]

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

[commit-bot: I haz the power, 2017-04-13 10:17:01]

Dry run: This issue passed the CQ dry run.

[Yang, 2017-04-13 10:25:03]

On 2017/04/13 09:17:32, jgruber wrote:

lgtm

[jgruber, 2017-04-13 10:31:15]

The CQ bit was checked by jgruber@chromium.org

[commit-bot: I haz the power, 2017-04-13 10:31:20]

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-04-13 10:33:09]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1492079475942420,
"parent_rev": "b30503387f4271d269d22715377c753d4110c8c5", "commit_rev":
"876e23c54d5fa2cb2a399632175d94e597da5330"}

[commit-bot: I haz the power, 2017-04-13 10:33:16]

Description was changed from

==========
[regexp] Fix incorrect DCHECK in FixSingleCharacterDisjunctions

The condition only applies in unicode mode, where any lone surrogates
are desugared into a character class (and will not be considered in this
optimization). Non-unicode mode treats lone surrogates exactly like
any other codepoint.

BUG=chromium:711092
==========

to

==========
[regexp] Fix incorrect DCHECK in FixSingleCharacterDisjunctions

The condition only applies in unicode mode, where any lone surrogates
are desugared into a character class (and will not be considered in this
optimization). Non-unicode mode treats lone surrogates exactly like
any other codepoint.

BUG=chromium:711092

Review-Url: https://codereview.chromium.org/2808403006
Cr-Commit-Position: refs/heads/master@{#44638}
Committed:
https://chromium.googlesource.com/v8/v8/+/876e23c54d5fa2cb2a399632175d94e597d...
==========

[commit-bot: I haz the power, 2017-04-13 10:33:17]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/v8/v8/+/876e23c54d5fa2cb2a399632175d94e597d...