Make CallSite constructor inaccessible from JS

Make CallSite constructor inaccessible from JS

This type is not supposed to be constructable by users. Internally, we
use CallSiteUtils::Construct to create CallSite objects; and we simply
map a thrower builtin as the public CallSite constructor.

R=yangguo@chromium.org
BUG=

Committed: https://crrev.com/8135caef32492b900f9d1d25276038d224b72ae1
Cr-Commit-Position: refs/heads/master@{#38234}

[jgruber, 2016-08-01 15:07:17]

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

[commit-bot: I haz the power, 2016-08-01 15:07:24]

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

[jgruber, 2016-08-01 15:09:10]

The kind of exception we throw is up for debate. I went with kUnsupported and
the generic Error class for now.

[jgruber, 2016-08-02 07:22:16]

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

[commit-bot: I haz the power, 2016-08-02 07:22:26]

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-08-02 07:48:11]

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

[commit-bot: I haz the power, 2016-08-02 07:48:13]

Dry run: This issue passed the CQ dry run.

[jgruber, 2016-08-02 08:19:28]

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

[commit-bot: I haz the power, 2016-08-02 08:19:37]

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

[jgruber, 2016-08-02 08:21:48]

https://codereview.chromium.org/2201823002/diff/60001/test/mjsunit/stack-trac...
File test/mjsunit/stack-traces.js (right):

https://codereview.chromium.org/2201823002/diff/60001/test/mjsunit/stack-trac...
test/mjsunit/stack-traces.js:404: Error.stackTraceLimit = oldStackTraceLimit;
Note: This fixes a few followup tests.

https://codereview.chromium.org/2201823002/diff/60001/test/mjsunit/stack-trac...
test/mjsunit/stack-traces.js:419: var err = new Error("abc");
Note: And the prepareStackTrace recursion test.

[commit-bot: I haz the power, 2016-08-02 08:44:11]

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

[commit-bot: I haz the power, 2016-08-02 08:44:11]

Dry run: This issue passed the CQ dry run.

[Yang, 2016-08-02 09:02:30]

lgtm.

[jgruber, 2016-08-02 09:06:50]

The CQ bit was checked by jgruber@chromium.org

[jgruber, 2016-08-02 09:06:50]

The patchset sent to the CQ was uploaded after l-g-t-m from yangguo@chromium.org
Link to the patchset: https://codereview.chromium.org/2201823002/#ps80001
(title: "Revert "CHECK invalid arguments to CallSite constructor"")

[commit-bot: I haz the power, 2016-08-02 09:06:57]

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-08-02 09:29:23]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-08-02 09:33:50]

Description was changed from

==========
Make CallSite constructor inaccessible from JS

This type is not supposed to be constructable by users. Internally, we
use CallSiteUtils::Construct to create CallSite objects; and we simply
map a thrower builtin as the public CallSite constructor.

R=yangguo@chromium.org
BUG=
==========

to

==========
Make CallSite constructor inaccessible from JS

This type is not supposed to be constructable by users. Internally, we
use CallSiteUtils::Construct to create CallSite objects; and we simply
map a thrower builtin as the public CallSite constructor.

R=yangguo@chromium.org
BUG=

Committed: https://crrev.com/8135caef32492b900f9d1d25276038d224b72ae1
Cr-Commit-Position: refs/heads/master@{#38234}
==========

[commit-bot: I haz the power, 2016-08-02 09:33:51]

Patchset 5 (id:??) landed as
https://crrev.com/8135caef32492b900f9d1d25276038d224b72ae1
Cr-Commit-Position: refs/heads/master@{#38234}