[parser] report invalid rest parameter errors in Arrow functions

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 7957 matching lines @@
                               NULL};
   RunParserSyncTest(sloppy_context_data, valid_data, kSuccess, NULL, 0,
                     always_flags, arraysize(always_flags));
   RunParserSyncTest(strict_context_data, valid_data, kError, NULL, 0,
                     always_flags, arraysize(always_flags));
   RunModuleParserSyncTest(strict_context_data, valid_data, kError, NULL, 0,
                           always_flags, arraysize(always_flags));
 }
 
 
 TEST(MiscSyntaxErrors) {

[adamk, 2016/02/01 23:15:59]

Not sure this test makes any sense, some grouping seems like a good idea. I
liked before when these were up with similar bits (also, much easier to review
if you don't delete patch sets).

[caitp (gmail), 2016/02/01 23:19:38]

I was just getting rid of ones that weren't quite doing what they were supposed
to be doing yet, sorry.

I'll put them back with the Arrow function syntax tests

+  // clang-format off
   const char* context_data[][2] = {
-      {"'use strict'", ""}, {"", ""}, {NULL, NULL}};
-  const char* error_data[] = {"for (();;) {}", NULL};
+    { "'use strict'", "" },
+    { "", "" },
+    { NULL, NULL }
+  };
+
+  const char* error_data[] = {
+    "for (();;) {}",
+
+      // crbug.com/582626
+    "{ NaN ,chA((evarA=new t ( l = !.0[((... co -a0([1]))=> greturnkf",
+
+    // Simplified case:
+    "(...rest - a) => b",

[adamk, 2016/02/01 23:15:59]

Please add a test for the multi-arg version too (to make sure the preparser also
throws the error in that case).

[caitp (gmail), 2016/02/01 23:19:37]

Acknowledged.

+    NULL
+  };
+  // clang-format on
 
   RunParserSyncTest(context_data, error_data, kError, NULL, 0, NULL, 0);
 }