Fix invalid syntax for JS setters in tests

Fix invalid syntax for JS setters in tests

ECMAScript setters require exactly one parameter.
We had a bunch of tests with zero parameters which
is not valid and should have been SyntaxErrors in
the first place.

BUG=383596
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=176451

[esprehn, 2014-06-18 17:59:37]

Can you explain the description why you had to add all these _ variables? Why is
that important?

lgtm, but this needs a better description.

[arv (Not doing code reviews), 2014-06-18 18:04:46]

On 2014/06/18 17:59:37, esprehn wrote:
> Can you explain the description why you had to add all these _ variables? Why
is
> that important?
> 
> lgtm, but this needs a better description.

_ is commonly used for don't care variables. At one point I had all of these as
value.

[esprehn, 2014-06-18 18:10:20]

On 2014/06/18 at 18:04:46, arv wrote:
> On 2014/06/18 17:59:37, esprehn wrote:
> > Can you explain the description why you had to add all these _ variables?
Why is
> > that important?
> > 
> > lgtm, but this needs a better description.
> 
> _ is commonly used for don't care variables. At one point I had all of these
as value.

Yeah I've done lots of that from my OCaml days :) I meant why does set foo() {}
not work and require the dummy arg. Thanks for updating the description.

[arv (Not doing code reviews), 2014-06-18 18:13:05]

On 2014/06/18 18:10:20, esprehn wrote:
> On 2014/06/18 at 18:04:46, arv wrote:
> > On 2014/06/18 17:59:37, esprehn wrote:
> > > Can you explain the description why you had to add all these _ variables?
> Why is
> > > that important?
> > > 
> > > lgtm, but this needs a better description.
> > 
> > _ is commonly used for don't care variables. At one point I had all of these
> as value.
> 
> Yeah I've done lots of that from my OCaml days :) I meant why does set foo()
{}
> not work and require the dummy arg. Thanks for updating the description.

set foo() {} is not valid ECMAScript. IE and Firefox both throws. The reason to
forbid it is to catch errors early. It is most likely an error to ignore the
value in an Put operation.

[arv (Not doing code reviews), 2014-06-18 18:13:09]

The CQ bit was checked by arv@chromium.org

[commit-bot: I haz the power, 2014-06-18 18:13:51]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/arv@chromium.org/339363002/1

[commit-bot: I haz the power, 2014-06-18 20:20:47]

Change committed as 176451