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

test/mjsunit/strong/function-arity.js

 // Copyright 2015 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Flags: --strong-mode --harmony-arrow-functions --harmony-reflect
 // Flags: --harmony-spreadcalls --harmony-rest-parameters --allow-natives-syntax
 
 'use strict';
 
 
@@ skipping 158 matching lines @@
 })();
 
 
 (function Constructor() {
   for (let genParams of [generateParams, generateParamsWithRest]) {
     for (let argumentCount = 0; argumentCount < 3; argumentCount++) {
       for (let parameterCount = 0; parameterCount < 3; parameterCount++) {
         let defs = [
           `'use strong';
           class C { constructor(${genParams(parameterCount)}) {} }`,
-          `'use strict';
-          class C {
-            constructor(${genParams(parameterCount, true)}) { 'use strong'; }
-          }`,
         ];
         for (let def of defs) {
           let calls = [
             `new C(${generateArguments(argumentCount)})`,
             `new C(${generateSpread(argumentCount)})`,
             `Reflect.construct(C, [${generateArguments(argumentCount)}])`,
           ];
           for (let call of calls) {
             let code = `${def}; ${call};`;
             if (argumentCount < parameterCount) {
@@ skipping 17 matching lines @@
           let defs = [
             `'use strong';
             class B {
               constructor(${genParams(parameterCount)}) {}
             }
             class C extends B {
               constructor() {
                 super(${genArgs(argumentCount)});
               }
             }`,
-            `'use strict';

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

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.

[conradw, 2015/09/01 16:55:54]

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

-            class B {
-              constructor(${genParams(parameterCount, true)}) { 'use strong'; }
-            }
-            class C extends B {
-              constructor() {
-                super(${genArgs(argumentCount)});
-              }
-            }`,
           ];
           for (let def of defs) {
             let code = `${def}; new C();`;
             if (argumentCount < parameterCount) {
               assertThrows(code, TypeError);
             } else {
               assertDoesNotThrow(code);
             }
           }
         }
       }
     }
   }
 })();
 
 
 (function DerivedConstructorDefaultConstructorInDerivedClass() {
   for (let genParams of [generateParams, generateParamsWithRest]) {
     for (let genArgs of [generateArguments, generateSpread]) {
       for (let argumentCount = 0; argumentCount < 3; argumentCount++) {
         for (let parameterCount = 0; parameterCount < 3; parameterCount++) {
           let defs = [
             `'use strong';
             class B {
               constructor(${genParams(parameterCount)}) {}
             }
             class C extends B {}`,
-            `'use strict';

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

Same here perhaps.

[conradw, 2015/09/01 16:55:54]

See above.

-            class B {
-              constructor(${genParams(parameterCount, true)}) { 'use strong'; }
-            }
-            class C extends B {}`,
           ];
           for (let def of defs) {
             let code = `${def}; new C(${genArgs(argumentCount)})`;
             if (argumentCount < parameterCount) {
               assertThrows(code, TypeError);
             } else {
               assertDoesNotThrow(code);
             }
           }
         }
@@ skipping 80 matching lines @@
           if (argumentCount < parameterCount) {
             assertThrows(code, TypeError);
           } else {
             assertDoesNotThrow(code);
           }
         }
       }
     }
   }
 })();