[regexp] Let RegExp.prototype.compile return this

[regexp] Let RegExp.prototype.compile return this

ES6 requires the compile method to return this:
www.ecma-international.org/ecma-262/6.0/#sec-regexp.prototype.compile

BUG=v8:5722, chromium:585775
Committed: https://crrev.com/f3b9d570cb79c7d1990d2ef11ce52ceb622b0e5a
Cr-Commit-Position: refs/heads/master@{#41705}

[jgruber, 2016-12-14 14:39:52]

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

[commit-bot: I haz the power, 2016-12-14 14:40:01]

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-14 15:05:58]

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

[commit-bot: I haz the power, 2016-12-14 15:05:58]

Dry run: This issue passed the CQ dry run.

[jgruber, 2016-12-14 15:12:49]

jgruber@chromium.org changed reviewers:
+ adamk@chromium.org, littledan@chromium.org

[jgruber, 2016-12-14 15:12:50]

This fixes one of the remaining ES6 incompatibilities:
https://kangax.github.io/compat-table/es6/

I got in touch with snapdeal.com customer service over twitter (since this
change breaks their site when loaded from google.com). Not sure whether it
actually got through to their web devs, let's hope for the best. Further details
in the associated bug reports.

Their site is also broken on every other recent major browser so I'd like to
land this in any case. PTAL

[Yang, 2016-12-14 15:14:59]

On 2016/12/14 15:12:50, jgruber wrote:
> This fixes one of the remaining ES6 incompatibilities:
> https://kangax.github.io/compat-table/es6/
> 
> I got in touch with http://snapdeal.com customer service over twitter (since
this
> change breaks their site when loaded from http://google.com). Not sure whether
it
> actually got through to their web devs, let's hope for the best. Further
details
> in the associated bug reports.
> 
> Their site is also broken on every other recent major browser so I'd like to
> land this in any case. PTAL

lgtm

[adamk, 2016-12-14 17:49:33]

lgtm

[Dan Ehrenberg, 2016-12-14 20:58:24]

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

[Dan Ehrenberg, 2016-12-14 20:58:25]

lgtm

[jgruber, 2016-12-15 07:26:22]

The CQ bit was checked by jgruber@chromium.org

[commit-bot: I haz the power, 2016-12-15 07:26:29]

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 07:28:26]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481786782168790, "parent_rev":
"7a3e1954763b65200a457047e59e0a23c1b6f317", "commit_rev":
"ec44a77e36f75c0185ded3ff138aea8c75a1e521"}

[commit-bot: I haz the power, 2016-12-15 07:28:31]

Description was changed from

==========
[regexp] Let RegExp.prototype.compile return this

ES6 requires the compile method to return this:
www.ecma-international.org/ecma-262/6.0/#sec-regexp.prototype.compile

BUG=v8:5722,chromium:585775
==========

to

==========
[regexp] Let RegExp.prototype.compile return this

ES6 requires the compile method to return this:
www.ecma-international.org/ecma-262/6.0/#sec-regexp.prototype.compile

BUG=v8:5722,chromium:585775

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

[commit-bot: I haz the power, 2016-12-15 07:28:32]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-15 07:29:45]

Description was changed from

==========
[regexp] Let RegExp.prototype.compile return this

ES6 requires the compile method to return this:
www.ecma-international.org/ecma-262/6.0/#sec-regexp.prototype.compile

BUG=v8:5722,chromium:585775

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

to

==========
[regexp] Let RegExp.prototype.compile return this

ES6 requires the compile method to return this:
www.ecma-international.org/ecma-262/6.0/#sec-regexp.prototype.compile

BUG=v8:5722,chromium:585775

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

[commit-bot: I haz the power, 2016-12-15 07:29:46]

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