Reviewers: Dmitry Lomov (chromium),
Message:
Committed patchset #1 (id:1) manually as 25272 (presubmit successful).
Description:
Classes: Add support for arguments in default constructor
This is currently done by generating a default constructor that looks
like this:
constructor() {
%DefaultConstructorSuperCall();
}
The a runtime function implements the logic which is pretty similar to
Runtime_Apply except that it uses the [[Prototype]] of the current
function.
This is the second try. The first failed because the test was using a
array that was too large for Function.prototype.apply.
Revert "Revert "Classes: Add support for arguments in default constructor""
This reverts commit 43aa7e541df56a132608b8b4217e9da84575e4f8.
BUG=v8:3672
LOG=Y
[email protected]
Committed: https://code.google.com/p/v8/source/detail?r=25272
Please review this at https://codereview.chromium.org/716853003/
Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files (+122, -11 lines):
M src/parser.cc
M src/runtime/runtime.h
M src/runtime/runtime-classes.cc
M src/runtime/runtime-object.cc
M test/mjsunit/harmony/classes.js
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
