[parser] report SyntaxError if rest parameter used in Setter MethodDefinition

src/messages.js

Index: src/messages.js
diff --git a/src/messages.js b/src/messages.js
index eb6e07fb07bdf6df49ead6f9288edf183127dae0..bb5789fd60a7ab84f3b964234b4cd03c4d3f9946 100644
--- a/src/messages.js
+++ b/src/messages.js
@@ -141,6 +141,7 @@ var kMessages = {
   super_constructor_call:        ["A 'super' constructor call may only appear as the first statement of a function, and its arguments may not access 'this'. Other forms are not yet supported."],
   duplicate_proto:               ["Duplicate __proto__ fields are not allowed in object literals"],
   param_after_rest:              ["Rest parameter must be last formal parameter"],
+  setter_rest_param:             ["Setter function argument must not be a rest parameter"],

[arv (Not doing code reviews), 2015/05/15 13:55:56]

Can you use messages.h instead since Yang is in the process of removing this
table from js.

[caitp (gmail), 2015/05/15 14:01:27]

I'd like to, but it's not obvious how to make the parser's ReportMessage
infrastructure use the new message stuff, which I think is why most of those
messages haven't been moved to that yet. The one SyntaxError which does use this
is part of a runtime function, which doesn't really apply to normal parsing.
Maybe yangguo could shed some light on this, I'll add them as a reviewer

   derived_constructor_return:    ["Derived constructors may only return object or undefined"],
   for_in_loop_initializer:       ["for-in loop variable declaration may not have an initializer."],
   for_of_loop_initializer:       ["for-of loop variable declaration may not have an initializer."],