[regexp] Fix matching of regexps that are both sticky and anchored at end.

[regexp] Fix matching of regexps that are both sticky and anchored at end.

V8 was applying incorrect optimization to them advancing the start position.

This would cause /foo$/y too match "barfoo", which it should not.

BUG=

Committed: https://crrev.com/29745ee927bf1f4cb3e0b0075d0a149575000f18
Cr-Commit-Position: refs/heads/master@{#41077}

[Vyacheslav Egorov (Google), 2016-11-17 13:26:51]

vegorov@google.com changed reviewers:
+ yangguo@chromium.org

[Vyacheslav Egorov (Google), 2016-11-17 13:26:52]

Found a small bug in irregexp while implementing similar sticky functionality in
Dart VM.

[Yang, 2016-11-17 13:28:28]

On 2016/11/17 13:26:52, Vyacheslav Egorov (Google) wrote:
> Found a small bug in irregexp while implementing similar sticky functionality
in
> Dart VM.

nice. lgtm.

[Vyacheslav Egorov (Google), 2016-11-17 13:33:56]

Description was changed from

==========
[regexp] Fix matching of regexps that are both sticky and anchored at end.

V8 was applying incorrect optimization to them advancing the start of the match
position.

This would cause /foo$/y too match "barfoo", which it should not.

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

to

==========
[regexp] Fix matching of regexps that are both sticky and anchored at end.

V8 was applying incorrect optimization to them advancing the start position.

This would cause /foo$/y too match "barfoo", which it should not.

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

[Vyacheslav Egorov (Google), 2016-11-17 13:34:00]

The CQ bit was checked by vegorov@google.com

[commit-bot: I haz the power, 2016-11-17 13:34:11]

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-11-17 13:58:27]

Description was changed from

==========
[regexp] Fix matching of regexps that are both sticky and anchored at end.

V8 was applying incorrect optimization to them advancing the start position.

This would cause /foo$/y too match "barfoo", which it should not.

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

to

==========
[regexp] Fix matching of regexps that are both sticky and anchored at end.

V8 was applying incorrect optimization to them advancing the start position.

This would cause /foo$/y too match "barfoo", which it should not.

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

[commit-bot: I haz the power, 2016-11-17 13:58:28]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-17 22:38:19]

Description was changed from

==========
[regexp] Fix matching of regexps that are both sticky and anchored at end.

V8 was applying incorrect optimization to them advancing the start position.

This would cause /foo$/y too match "barfoo", which it should not.

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

to

==========
[regexp] Fix matching of regexps that are both sticky and anchored at end.

V8 was applying incorrect optimization to them advancing the start position.

This would cause /foo$/y too match "barfoo", which it should not.

BUG=

Committed: https://crrev.com/29745ee927bf1f4cb3e0b0075d0a149575000f18
Cr-Commit-Position: refs/heads/master@{#41077}
==========

[commit-bot: I haz the power, 2016-11-17 22:38:21]

Patchset 3 (id:??) landed as
https://crrev.com/29745ee927bf1f4cb3e0b0075d0a149575000f18
Cr-Commit-Position: refs/heads/master@{#41077}