[strong] Class constructor bodies cannot contain "use strong" directive

[strong] Class constructor bodies cannot contain "use strong" directive

Since the constructor is also the class object itself, allowing it to
retroactively become a strong object would have unintuitive consequences
wrt the strength of the other functions of the class, and whether instances
would be considered instances of a strong class.

BUG=v8:3956
LOG=N

Committed: https://crrev.com/05f01b3f8e2fbb2eac0f4495d165f928b2679570
Cr-Commit-Position: refs/heads/master@{#30519}

[conradw, 2015-08-26 15:39:41]

conradw@chromium.org changed reviewers:
+ rossberg@chromium.org

[conradw, 2015-08-26 15:39:42]

PTAL

[conradw, 2015-09-01 08:09:35]

conradw@chromium.org changed reviewers:
+ caitpotter88@gmail.com, mstarzinger@chromium.org

[conradw, 2015-09-01 08:09:35]

Adding some other reviewers since rossberg is at ICFP, this is blocking a couple
of other things.

[rossberg, 2015-09-01 16:52:40]

LGTM modulo comment

https://codereview.chromium.org/1314203002/diff/40001/test/mjsunit/strong/fun...
File test/mjsunit/strong/function-arity.js (left):

https://codereview.chromium.org/1314203002/diff/40001/test/mjsunit/strong/fun...
test/mjsunit/strong/function-arity.js:220: `'use strict';
I don't think you want to delete this test altogether. It would still be good to
have test for a strict class deriving from a strong one, though you'll have to
write it slightly differently.

https://codereview.chromium.org/1314203002/diff/40001/test/mjsunit/strong/fun...
test/mjsunit/strong/function-arity.js:256: `'use strict';
Same here perhaps.

[conradw, 2015-09-01 16:55:55]

https://codereview.chromium.org/1314203002/diff/40001/test/mjsunit/strong/fun...
File test/mjsunit/strong/function-arity.js (left):

https://codereview.chromium.org/1314203002/diff/40001/test/mjsunit/strong/fun...
test/mjsunit/strong/function-arity.js:220: `'use strict';
On 2015/09/01 16:52:40, rossberg wrote:
> I don't think you want to delete this test altogether. It would still be good
to
> have test for a strict class deriving from a strong one, though you'll have to
> write it slightly differently.

My impression was that it would need to be deleted anyway, since we're also
preventing weak classes from inheriting from strong ones.

https://codereview.chromium.org/1314203002/diff/40001/test/mjsunit/strong/fun...
test/mjsunit/strong/function-arity.js:256: `'use strict';
On 2015/09/01 16:52:40, rossberg wrote:
> Same here perhaps.

See above.

[conradw, 2015-09-01 16:58:23]

The CQ bit was checked by conradw@chromium.org

[commit-bot: I haz the power, 2015-09-01 16:58:38]

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

[commit-bot: I haz the power, 2015-09-01 18:29:31]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-09-01 18:29:42]

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