Object.prototype.__define{G,S}etter__ with non-callable second parameter should throw TypeError instead of SyntaxError

[Mathias Bynens]

For compatibility with other ECMAScript engines, `Object.prototype.__defineGetter__` must behave as described here: http://mathias.html5.org/specs/javascript/#object.prototype.__definegetter__

JavaScriptCore has a minor bug where it throws a SyntaxError instead of a TypeError in cases like the following:

    var object = {};
    object.__defineGetter__('foo', '_');

Online tests: http://mathias.html5.org/tests/javascript/object/

[Mathias Bynens]

So obvious I forgot to mention it, but the same goes for `Object.prototype.__defineSetter__`.

[Gavin Barraclough]

Yup - syntax errors should all be early errors, doesn't sounds appropriate here.  We should make this a type error.  Thanks for the bug report!

[Gavin Barraclough]

Created attachment 162885 [details]
Fix

[Gavin Barraclough]

Fixed in r127930