[Intl] create new instances when new.target is undefined

[Intl] create new instances when new.target is undefined

BUG=v8:4360
LOG=N
R=littledan@chromium.org

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

[caitp (gmail), 2015-11-11 19:36:30]

PTAL when you get some time, since you've been poking around ecma402 stuff
lately

[caitp (gmail), 2015-11-11 21:17:05]

Description was changed from

==========
[Intl] create new instances when new.target is undefined

Avoid relying on whether or not `this` is a specific value, as there are
observable side effects to this.

BUG=v8:4360
LOG=N
R=littledan@chromium.org
==========

to

==========
[Intl] create new instances when new.target is undefined

BUG=v8:4360
LOG=N
R=littledan@chromium.org
==========

[Dan Ehrenberg, 2015-11-12 19:58:16]

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

[commit-bot: I haz the power, 2015-11-12 19:58:21]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1440593003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1440593003/1

[Dan Ehrenberg, 2015-11-12 20:29:35]

lgtm

Looks good, seems to match the spec by my reading. Tests look great too, but I
wouldn't bother writing them in such detail next time for these edge cases.
Test262 already gets these, and we run it; I don't see a strong need to
duplicate it for an obscure case like this.

[Dan Ehrenberg, 2015-11-12 20:29:35]

lgtm

Looks good, seems to match the spec by my reading. Tests look great too, but I
wouldn't bother writing them in such detail next time for these edge cases.
Test262 already gets these, and we run it; I don't see a strong need to
duplicate it for an obscure case like this.

[commit-bot: I haz the power, 2015-11-12 20:34:57]

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

[commit-bot: I haz the power, 2015-11-12 20:34:58]

Dry run: This issue passed the CQ dry run.

[caitp (gmail), 2015-11-12 21:16:11]

The CQ bit was checked by caitpotter88@gmail.com

[caitp (gmail), 2015-11-12 21:16:11]

The patchset sent to the CQ was uploaded after l-g-t-m from
littledan@chromium.org
Link to the patchset: https://codereview.chromium.org/1440593003/#ps20001
(title: "Remove unwanted tests")

[commit-bot: I haz the power, 2015-11-12 21:16:24]

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

[commit-bot: I haz the power, 2015-11-12 21:46:11]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-11-14 23:21:06]

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

[Dan Ehrenberg, 2015-11-24 01:54:30]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/1473493003/ by littledan@chromium.org.

The reason for reverting is: This breaks backwards compatibility by disallowing
call. Web application authors have noticed the breakage.
https://github.com/tc39/ecma402/issues/57.