[regexp] fix zero-length matches for RegExp.prototype.@@split.

[regexp] fix zero-length matches for RegExp.prototype.@@split.

BUG=v8:4717
LOG=N

Committed: https://crrev.com/aff7bd54beb74a6effba330ea940d3ee31d9717f
Cr-Commit-Position: refs/heads/master@{#33706}

[Yang, 2016-02-03 11:32:05]

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

[Toon Verwaest, 2016-02-03 13:11:32]

lgtm with comment

https://codereview.chromium.org/1663543003/diff/1/src/js/regexp.js
File src/js/regexp.js (right):

https://codereview.chromium.org/1663543003/diff/1/src/js/regexp.js#newcode300
src/js/regexp.js:300: if (index + 1 > subject.length) return false;
if (index + 1 >= subject.length)

[Yang, 2016-02-03 14:24:24]

The CQ bit was checked by yangguo@chromium.org

[Yang, 2016-02-03 14:24:24]

The patchset sent to the CQ was uploaded after l-g-t-m from
verwaest@chromium.org
Link to the patchset: https://codereview.chromium.org/1663543003/#ps20001
(title: "address comment")

[commit-bot: I haz the power, 2016-02-03 14:24:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1663543003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1663543003/20001

[Yang, 2016-02-03 14:29:29]

Thanks for catching this. Fixed.

[commit-bot: I haz the power, 2016-02-03 14:48:38]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-03 14:49:15]

Description was changed from

==========
[regexp] fix zero-length matches for RegExp.prototype.@@split.

BUG=v8:4717
LOG=N
==========

to

==========
[regexp] fix zero-length matches for RegExp.prototype.@@split.

BUG=v8:4717
LOG=N

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

[commit-bot: I haz the power, 2016-02-03 14:49:16]

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