[regexp] Consider surrogate pairs when optimizing disjunctions

[regexp] Consider surrogate pairs when optimizing disjunctions

RationalizeConsecutiveAtoms optimizes ab|ac|az to a(?:b|c|d).
Ensure that this optimization does not split surrogate pairs in unicode
mode.

BUG=chromium:641091
Review-Url: https://codereview.chromium.org/2813893002
Cr-Commit-Position: refs/heads/master@{#44599}
Committed: https://chromium.googlesource.com/v8/v8/+/4635572471b28a38f04307fabc69d5e1802d1049

[jgruber, 2017-04-11 14:58:04]

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

[commit-bot: I haz the power, 2017-04-11 14:58:06]

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-11 15:02:24]

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

[commit-bot: I haz the power, 2017-04-11 15:21:21]

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

[commit-bot: I haz the power, 2017-04-11 15:21:22]

Dry run: This issue passed the CQ dry run.

[Yang, 2017-04-12 06:38:10]

On 2017/04/11 15:02:24, jgruber wrote:

lgtm.

[jgruber, 2017-04-12 07:53:09]

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

[commit-bot: I haz the power, 2017-04-12 07:53: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-12 07:55:33]

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

[commit-bot: I haz the power, 2017-04-12 07:55:42]

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-12 08:21:19]

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

[commit-bot: I haz the power, 2017-04-12 08:21:20]

Dry run: This issue passed the CQ dry run.

[jgruber, 2017-04-12 08:42:37]

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

[commit-bot: I haz the power, 2017-04-12 08:42:46]

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

[Yang, 2017-04-12 09:06:00]

On 2017/04/12 06:38:10, Yang wrote:
> On 2017/04/11 15:02:24, jgruber wrote:
> 
> lgtm.

still lgtm.

[jgruber, 2017-04-12 09:06:57]

The CQ bit was unchecked by jgruber@chromium.org

[jgruber, 2017-04-12 09:07:04]

The CQ bit was checked by jgruber@chromium.org

[commit-bot: I haz the power, 2017-04-12 09:07:14]

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-12 09:09:12]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1491988024276520,
"parent_rev": "483812d46cf96e26500b93b5b3d1cb9737236b2b", "commit_rev":
"4635572471b28a38f04307fabc69d5e1802d1049"}

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

Description was changed from

==========
[regexp] Consider surrogate pairs when optimizing disjunctions

RationalizeConsecutiveAtoms optimizes ab|ac|az to a(?:b|c|d).
Ensure that this optimization does not split surrogate pairs in unicode
mode.

BUG=chromium:641091
==========

to

==========
[regexp] Consider surrogate pairs when optimizing disjunctions

RationalizeConsecutiveAtoms optimizes ab|ac|az to a(?:b|c|d).
Ensure that this optimization does not split surrogate pairs in unicode
mode.

BUG=chromium:641091

Review-Url: https://codereview.chromium.org/2813893002
Cr-Commit-Position: refs/heads/master@{#44599}
Committed:
https://chromium.googlesource.com/v8/v8/+/4635572471b28a38f04307fabc69d5e1802...
==========

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

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/v8/v8/+/4635572471b28a38f04307fabc69d5e1802...