Disallow yield in computed property names of class expressions in params

test/cctest/test-parsing.cc

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 6763 matching lines @@
 
   // Because classes are always in strict mode, these are always errors.
   const char* always_error_param_data[] = {
     "x = class extends (yield) { }",
     "x = class extends f(yield) { }",
     "x = class extends (null, yield) { }",
     "x = class extends (a ? null : yield) { }",
     "[x] = [class extends (a ? null : yield) { }]",
     "[x = class extends (a ? null : yield) { }]",
     "[x = class extends (a ? null : yield) { }] = [null]",
+    "x = class { [yield]() { } }",
+    "x = class { static [yield]() { } }",

[Dan Ehrenberg, 2016/05/04 22:36:34]

Could you add a test which requires deeper recursion to get to, e.g., x = class
{ static [yield, 1]() { } }? Should work fine with our expression classifier,
just thinking about future refactorings.

[adamk, 2016/05/04 22:40:39]

Done.

     NULL
   };
   // clang-format on
 
   RunParserSyncTest(sloppy_function_context_data, parameter_data, kSuccess);
   RunParserSyncTest(sloppy_arrow_context_data, parameter_data, kSuccess);
 
   RunParserSyncTest(strict_function_context_data, parameter_data, kError);
   RunParserSyncTest(strict_arrow_context_data, parameter_data, kError);
 
@@ skipping 659 matching lines @@
     "for (const x = 0 in {});",
     "for (let x = 0 in {});",
     NULL
   };
   // clang-format on
 
   static const ParserFlag always_flags[] = {kAllowHarmonyForIn};
   RunParserSyncTest(context_data, error_data, kError, nullptr, 0, always_flags,
                     arraysize(always_flags));
 }