Move ParsePostfixExpression into ParserBase.

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 2314 matching lines @@
   const char* context_data[][2] = {
     {"", ""},
     {"var f =", ""},
     { NULL, NULL }
   };
 
   const char* statement_data[] = {
     "new foo bar",
     "new ) foo",
     "new ++foo",
     // TODO(marja): Activate this test once the preparser checks correctly.

[Michael Starzinger, 2014/03/21 09:24:37]

Is this test also fixed by this change?

[marja, 2014/03/21 09:28:35]

Ah, yes. Re-enabled it.

     // "new foo ++",
     NULL
   };
 
   RunParserSyncTest(context_data, statement_data, kError);
 }
 
 
 TEST(StrictObjectLiteralChecking) {
   const char* strict_context_data[][2] = {
@@ skipping 175 matching lines @@
   RunParserSyncTest(sloppy_context_data, sloppy_statement_data, kSuccess);
 
   RunParserSyncTest(strict_context_data, good_statement_data, kSuccess);
   RunParserSyncTest(sloppy_context_data, good_statement_data, kSuccess);
 
   RunParserSyncTest(strict_context_data, bad_statement_data, kError);
   RunParserSyncTest(sloppy_context_data, bad_statement_data, kError);
 }
 
 
-TEST(ErrorInvalidLeftHandSide) {
+TEST(InvalidLeftHandSide) {
   const char* assignment_context_data[][2] = {
     // {"", " = 1;"},
     // {"\"use strict\"; ", " = 1;"},
     { NULL, NULL }
   };
 
   const char* prefix_context_data[][2] = {
     {"++", ";"},
     {"\"use strict\"; ++", ";"},
     {NULL, NULL},
@@ skipping 44 matching lines @@
 
   RunParserSyncTest(assignment_context_data, good_statement_data, kSuccess);
   RunParserSyncTest(assignment_context_data, bad_statement_data_common, kError);
   RunParserSyncTest(assignment_context_data, bad_statement_data_for_assignment,
                     kError);
 
   RunParserSyncTest(prefix_context_data, good_statement_data, kSuccess);
   RunParserSyncTest(prefix_context_data, bad_statement_data_common, kError);
 
   RunParserSyncTest(postfix_context_data, good_statement_data, kSuccess);
-  // TODO(marja): This doesn't work yet.
-  // RunParserSyncTest(postfix_context_data, bad_statement_data_common, kError);
+  RunParserSyncTest(postfix_context_data, bad_statement_data_common, kError);
 }