Make invalid LHSs a parse-time (reference) error

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 2200 matching lines @@
     "new foo.bar();",
     "new foo.bar.baz;",
     "new foo.bar().baz;",
     "new foo[bar];",
     "new foo[bar]();",
     "new foo[bar][baz];",
     "new foo[bar]()[baz];",
     "new foo[bar].baz(baz)()[bar].baz;",
     "new \"foo\"",  // Runtime error
     "new 1",  // Runtime error
-    "new foo++",

[Michael Starzinger, 2014/03/17 09:28:17]

IIUC, this should throw an "invalid LHS" reference error now. Can we move it
into the ErrorsNewExpression test case? I assume the reason for removing it was
that parser and pre-parser don't agree (yet). In this case I would propose
leaving a TODO to add that test again once they do.

[rossberg, 2014/03/17 10:12:32]

Done.

     // This even runs:
     "(new new Function(\"this.x = 1\")).x;",
     "new new Test_Two(String, 2).v(0123).length;",
     NULL
   };
 
   RunParserSyncTest(context_data, statement_data, kSuccess);
 }
 
 
@@ skipping 139 matching lines @@
 
   const char* statement_data[] = {
     statement,
     NULL
   };
 
   // The test is quite slow, so run it with a reduced set of flags.
   static const ParserFlag empty_flags[] = {kAllowLazy};
   RunParserSyncTest(context_data, statement_data, kError, empty_flags, 1);
 }